89973 - color picker doesn't show correct color on chip or editfield

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect)

Description

[Kathleen :Brade]

In Composer the color picker doesn't show the correct color on the color chip on 
the toolbar.  Also, the editfield doesn't always show a value when it should.

To reproduce:
  type some text in Composer
  select the text
  choose the text color picker
  enter: #eeeeee
  click ok
Notice that the color picker shows a white color chip but the text is a very 
light gray.  
  go back to color picker
Notice that the edit field is empty; it should show "#eeeeee"

Comment 1

[Charles Manske]

This problem occurs when the color typed in is *not* in the array of color
swatches.

Comment 2

[Charles Manske]

This bug occurs because of recent change in how color picker is intialized.
We should call "initColor()" else we can't use a color that isn't in the 
color swatch array! (Not a good thing.)
Fix is very simple. Nominating to fix for 6.1.

r=sfraser

Comment 3

[Charles Manske]

Attachment: Fix initialization of colorpicker color

Comment 4

[Charles Manske]

Per discussion with beppe, setting nsBranch.

Comment 5

[kinmoz]

sr=[e-mail]

Comment 6

[Simon Fraser [no longer active]]

I didn't r= this yet. I said I *would*, not that I *had*

Comment 7

[Simon Fraser [no longer active]]

ok, r=sfraser

Comment 8

[Charles Manske]

checked into trunk.

Comment 9

[rubydoo123]

adding nsBranch+

Comment 10

[Charles Manske]

PDT+ approved by Steve Elmer.

Comment 11

[Charles Manske]

Checked into branch

Comment 12

[sujay]

verified on 7/11 trunk.

now I will verify on the branch.

removing vtrunk keyword

Comment 13

[sujay]

verified on 7/13 branch on windows.

marking verified-fixed.