Opened 9 years ago

Last modified 6 years ago

#6581 confirmed Bug

Invalid background color triggers js error in IE and erratic behavior in others

Reported by: Dinu Owned by:
Priority: Normal Milestone:
Component: UI : Dialogs Version: 3.0
Keywords: oracle Cc:

Description

To reproduce:

  • insert table
  • cell properties
  • in background field, enter invalid color "foo"
  • click ok

Results:

  • in IE: "Invalid property value" error, ok button is rendered useless; an anecdotic form submission happened but can't reproduce
  • in FF and others: value seems to be ignored (I think this is wrong, invalid value should be used regardless as long as it doesn't break CSS), however element's style does get rewritten and dirty flag gets set

Expected behavior (I presume):

  • Either update the invalid value regardless (don't try to parse to rgb(...))
  • Or, provide a remark to the user and don't close the dialog and don't update any document contents


Secondary:

  • A valid value without "#" is invalid just the same: "111111"; for convenience, a # should be implied when it can be hex-parsed ("111","111111")

Change History (3)

comment:1 Changed 9 years ago by Dinu

Update: having a space anywhere in the color breaks format; a trim should be used to clear before and after space. Test: " #111111", "#111111 "

comment:2 Changed 9 years ago by Garry Yao

Status: newconfirmed
Version: 3.4.13.0

comment:3 Changed 6 years ago by Jakub Ś

Keywords: oracle added

#10647 was marked as duplicate

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy