[Webkit-unassigned] [Bug 28978] Color swatches should show before the color text

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28978


Timothy Hatcher <timothy at hatcher.name> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #39075|                            |review?
               Flag|                            |




--- Comment #1 from Timothy Hatcher <timothy at hatcher.name>  2009-09-04 10:48:10 PDT ---
Created an attachment (id=39075)
 --> (https://bugs.webkit.org/attachment.cgi?id=39075)
Proposed Change

shift the swatch. Also makes multiple swatches in the single property toggle
the color format.

https://bugs.webkit.org/show_bug.cgi?id=28978

Reviewed by NOBODY (OOPS!).

* inspector/front-end/StylesSidebarPane.js:
(WebInspector.StylePropertyTreeElement.prototype.updateTitle): Some refactoring
to consolidate
the processing we do on the property value. This eliminated the old nickname
code, since the new
WebInspector.Color class handles this. We could also simplify the color regex
since more
is handled by the Color class. Also no longer uses innerHTML for the linkify
code.
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.processValue):
Helper function to
process a value given a regex and processor functions.
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.linkifyURL): Make
an anchor for the
URL with "url()" syntax surrounding it.
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.processColor):
Makes a color, if
there was an exception just return a text node.
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.processColor.changeColorDisplay.changeTo):
Moved from later in the file.
(WebInspector.StylePropertyTreeElement.prototype.updateTitle.processColor.changeColorDisplay):
Ditto.
* inspector/front-end/inspector.css:
---
 3 files changed, 142 insertions(+), 127 deletions(-)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.