Closed
Bug 40204
Opened 25 years ago
Closed 24 years ago
Composer color picker swatches on toolbar don't update correctly
Categories
(Core :: DOM: Editor, defect, P3)
Tracking
()
VERIFIED
FIXED
M18
People
(Reporter: bijals, Assigned: cmanske)
References
Details
(Whiteboard: [nsbeta3+][p:3])
Steps:
1) Open a blank or an existing document
2) Type new text and select text on document
3) Use the color picker to change the text color
Actual Results: Color picker icon changes, but selected text stays the same
color. Type more and new text does not change to the new color picked.
Expected Results: Selected text color changes to new color as indicated by Color
picker icon. This should work for both background and foreground.
Reporter | ||
Comment 1•25 years ago
|
||
Build Platform/Date: WinNT using 2000052208
Reporter | ||
Comment 2•25 years ago
|
||
I opened the Composer tabs bar and clicked on a few tabs. Surprisingly, the
color did apply as expected but does not get reflected unless I click on these
tabs.
Since this Composer tab bar will be closed by default, we need the colors
applying without requiring the end user to go into this tab mode.
Assignee | ||
Comment 4•25 years ago
|
||
The fact that the color appears correct after changing modes indicates that
this is a layout bug, since changing modes forces a repaint.
Assignee | ||
Comment 5•25 years ago
|
||
Text color (upper left color button) seems to work fine. It's the background
color that doesn't seem to work. But for both, I have to click 2 or 3 times
before I event get the popup!
Marc has dealt with the problem before (bug 11491), although I'm not sure if
it's the same problem since now resizing the window doesn't trigger the
repaint of background. Note that changing "edit modes" loads/unloads style
sheets, which seems to trigger the background repaint.
Setting background color seems to be a dogfood issue.
Putting on [nsbeta2+][6/15] radar.
Whiteboard: [nsbeta2+][6/15][dogfood-]
Comment 7•25 years ago
|
||
I have been trying to duplicate this with my new build (pulled source this
morning) and it is working fine. I do have some unrelated chagnes in my tree as
well, so I'll try with a spanking-clean build and see what is up. It sure looks
like a WORKSFORME... Unless I am doing it wrong. The steps seem simple enough
though.
This may be related to a fix I had to put in for a regression on background
colors being set via the BGCOLOR attribute on the body. I just checked the fix
in so if you are setting the background color like that then this is probably
fixed. Might want to grab /mozilla/layout/html/base/src/nsFrameManager.cpp and
try it, or just test it with tomorrow's build.
Assignee | ||
Comment 8•25 years ago
|
||
I just updated Marc's latest checkin and that fixes this.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
almost fixed...
color picker toolbar icon does not change to the new color unless you glide
your mouse cursor over it...then it changes to the new color...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 10•25 years ago
|
||
I don't think you want me trying to fix that! Back to Charles...
Assignee: attinasi → cmanske
Status: REOPENED → NEW
Assignee | ||
Comment 11•25 years ago
|
||
Updating the color swatch on the toolbar is a totally different issue.
There are general problems with updating CSS styles and attributes in toolbars;
probably belong to the XUL team, but they are CSS-driven, so Marc may not
be totally off the hook!
Status: NEW → ASSIGNED
Summary: Composer color picker does not change color → Composer color picker swatches on toolbar don't update correctly
Assignee | ||
Comment 12•24 years ago
|
||
Beppe thought this was a good task for Tony
Assignee: cmanske → anthonyd
Status: ASSIGNED → NEW
Assignee | ||
Comment 13•24 years ago
|
||
I'd like to keep this bug. I'm sure I can find others to pass
on to Tony!
This is blocked now because we can't get the color of the text in the page
at the selection.
Removing all nsbeta2 keywords and status -- we'll tackle it for beta3.
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M16 → M17
Comment 14•24 years ago
|
||
m18 - nsbeta3+
Comment 15•24 years ago
|
||
*** Bug 45905 has been marked as a duplicate of this bug. ***
Comment 16•24 years ago
|
||
*** Bug 45905 has been marked as a duplicate of this bug. ***
Comment 17•24 years ago
|
||
*** Bug 45905 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 19•24 years ago
|
||
*** Bug 49705 has been marked as a duplicate of this bug. ***
Comment 20•24 years ago
|
||
I did not see this on today's builds
windows & linux 2000-08-22-06-M18 no mac build yet
Assignee | ||
Comment 21•24 years ago
|
||
checked in 8/22.
Just one small problem:
Click cursor to locate caret in text ("collapsed selection")
Select a differnt text color using colorpicker, but *dont type anything*
(if you did, it would be in the new color, so color picker button color
shows the new color correctly.) Instead, move caret using left arrow into
previous text. The color button color should change to the color of that
text, but it doesn't. I've discussed this with jfrancis and he thinks
it's a bug in his code -- we'll follow up with a separate bug on him.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•