Closed Bug 115694 Opened 23 years ago Closed 23 years ago

Composer colorpicker widget on toolbar doesn't show correct "text" attribute color on <body> element

Categories

(SeaMonkey :: Composer, defect)

Product:
SeaMonkey
Component:
Composer
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.8

People

(Reporter: cmanske, Assigned: cmanske)

References

Details

Attachments

(1 file, 1 obsolete file)

Update: Trigger UI update only if color value changed
1.81 KB, patch
slogan
: review+
hewitt
: superreview+
Details | Diff | Splinter Review
1. Launch Composer 2. Use Format | Page Colors and Background dialog 3. Click on "Use Custom Colors" radio and change the "Normal Text" color to something other than black. 4. Change Background color as well. 5. Click Ok to apply color. Any text in the page that is "default" (no <font color=.. used) and the background does change correctly in the page, but not on the toolbar swatches. Two problems: 1. When caret is in the 'default' (normal) text, the color swatch in the toolbar never shows the actual color set on the "text" attribute by the dialog. 2. The color swatch for the background does show correctly, but only after you move the caret with arrow key or clicking somewhere.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
Attached patch Fix (obsolete) — — Splinter Review
Keywords: patch, review
Whiteboard: FIX IN HAND, need r=,sr=
Attachment #62266 - Attachment is obsolete: true
Comment on attachment 62270 [details] [diff] [review] Update: Trigger UI update only if color value changed r=syd
Attachment #62270 - Flags: review+
Whiteboard: FIX IN HAND, need r=,sr= → FIX IN HAND, need sr=
*** Bug 100327 has been marked as a duplicate of this bug. ***
*** Bug 100408 has been marked as a duplicate of this bug. ***
Comment on attachment 62270 [details] [diff] [review] Update: Trigger UI update only if color value changed sr=hewitt
Attachment #62270 - Flags: superreview+
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Keywords: patch, review
Resolution: --- → FIXED
Whiteboard: FIX IN HAND, need sr=
Looks good to me on Win 2k using trunk build 2002010803. Verified.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: