Closed
Bug 222799
Opened 21 years ago
Closed 20 years ago
Options dialog assumes white background/black text colours
Categories
(Firefox :: Settings UI, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: greenreaper, Assigned: bugzilla)
References
Details
(Keywords: fixed-aviary1.0)
Attachments
(2 files, 2 obsolete files)
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20031007 Firebird/0.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20031007 Firebird/0.7
The Firebird options panel inherits the default Windows colours for dialogs.
Usually these include white window backgrounds and black window text. For me,
these are dark blue backgrounds and white text. This is not a problem for most
things, but the selected and mouseover background colours for the sections are
light, and they do *not* force the text to be black, so the selected/mouseover'd
section text is barely readable. Similarly, the selected group of options in the
right panel has a light yellow background, and does not force black text, thus
causing the options to be unreadable.
Reproducible: Always
Steps to Reproduce:
1. Set system colours to white text/dark background (either manually or using a
skin that has them)
2. Start Mozilla and go to the Options dialog
3. Expand an option on the right
Actual Results:
The expanded options were not easily readable, and mousing over an item on the
left caused it to be unreadable.
Expected Results:
If the options dialog is going to force a light background on some things, it
should stop assuming that the background is going to be white and that text is
going to be black and instead force these things as well (if you're forcing it,
at least force everything! ;-). Alternatively it could use a different *system*
colour to do so, but it should be careful and choose a colour that will never be
unreadable on another.
Current skin: Blackcomb Dark, included in WindowBlinds (www.windowblinds.net0
default install.
Reporter | ||
Comment 1•21 years ago
|
||
Comment 2•21 years ago
|
||
Confirming.
Comment 3•21 years ago
|
||
*** Bug 224730 has been marked as a duplicate of this bug. ***
Comment 4•21 years ago
|
||
*** Bug 231024 has been marked as a duplicate of this bug. ***
Comment 5•21 years ago
|
||
I was having the same problem. Apparently, it seemes to take the system color I
use as a window background, which is white, with the colors I use for menu text
(also white). Unreadable, of course, caused by the fact that menu text never
appears on the window background. They should use a scheme that does, such as
the system's window background and text colors (usually white and black, so no
change for most, but it would always be readable).
Updated•21 years ago
|
Flags: blocking0.9?
Comment 6•21 years ago
|
||
not critical for 0.9, putting on 1.0 radar
Flags: blocking1.0?
Flags: blocking0.9?
Flags: blocking0.9-
Comment 7•21 years ago
|
||
> not critical for 0.9, putting on 1.0 radar
Maybe, but this really isn't a difficult bug. I don't see why a two minute fix
couldn't be made for 0.9.
Comment 8•21 years ago
|
||
"we're accepting patches" is always going to be true. If someone attaches a
patch I'll take a look, but this isn't a high-visibility thing to deal with
ahead of other things.
There are hundreds of open bugs, unless its really critical I doubt that it'll
happen for a release that's hopefully less than three weeks away
Comment 9•21 years ago
|
||
A two minute fix that maintains an attractive visual appearance for the vast
majority of people that use non-clashing color schemes?
Just using system colors here isn't going to fly.
Comment 10•21 years ago
|
||
Oh, you were talking about the right section of the panel. The problem I have
that I described earlier (which was partially inaccurate anyway) concerned the
left side. I attached a screen shot with it, and a comparison to notepad, to
show the system colors used. All this was on Windows 98.
Currently,on Windows 98 at least, the system colors for the left panel seem to
go like this:
Background: the "Window" setting for background, as used in, for example, Notepad.
Text Labels: the "3D Objects" text setting. As this text never occurs naturally
on top of the 'Window' backgrounds, problems can occur. I would suggest using
the text color set under 'Windows', which always works. I don't know, however,
how system colors work on W XP and other OS's is equivalent. If someone could
test that for me...
Comment 11•21 years ago
|
||
Updated•21 years ago
|
Flags: blocking1.0? → blocking1.0-
Comment 12•21 years ago
|
||
*** Bug 244992 has been marked as a duplicate of this bug. ***
Comment 13•21 years ago
|
||
OK, I have a patch for the problem with the left panel. I added a semicolon and
color setting in pref.css here. It solves it on my PC.
.buttonBoxButton {
-moz-appearance: none;
border: 1px solid transparent;
background-color: transparent;
margin-left: 1px;
margin-right: 1px;
list-style-image: url("chrome://browser/skin/pref/Options.png");
color: WindowText;
}
Working on the privacy panel.
Comment 14•21 years ago
|
||
Changes: Specified text set to "WindowText" on left panel, and set to 'Black'
in highlighted area in privacy panel. Needs testing
Comment 15•21 years ago
|
||
To test, alter system colors to make left and right panels unreadable, then
unzip classic.jar, replace pref.jar with the attachment I made, rezip and
replace, and restart Firefox.
Comment 16•21 years ago
|
||
Comment on attachment 153934 [details] [diff] [review]
Proposed patch.
Adds two properties to Pref.css in classic.jar to fix visibility problems in
the options and privacy panels for people with unusual system colors. Bug
222799
Attachment #153934 -
Flags: review?(firefox)
Comment 17•21 years ago
|
||
Comment on attachment 153934 [details] [diff] [review]
Proposed patch.
This will never get reviewed because it is not a patch. See
http://www.mozilla.org/build/unix-cheatsheet.html#makePatch
Attachment #153934 -
Flags: review?(firefox)
Comment 18•21 years ago
|
||
(In reply to comment #17)
> (From update of attachment 153934 [details] [diff] [review])
> This will never get reviewed because it is not a patch. See
> http://www.mozilla.org/build/unix-cheatsheet.html#makePatch
>
Sorry, I didn't know. Still don't. Those are the fixes I made; if anyone who
knows how to register that knows how, go ahead.
Comment 19•21 years ago
|
||
Attachment #153934 -
Attachment is obsolete: true
Comment 20•21 years ago
|
||
Comment on attachment 154281 [details] [diff] [review]
patch
Based on comment 8, I ask Mike Connor for review.
Attachment #154281 -
Flags: review?(mconnor)
Updated•20 years ago
|
Attachment #154281 -
Flags: review?(mconnor) → review?(vladimir)
Comment 21•20 years ago
|
||
Can someone confirm the fix for this?
Comment 22•20 years ago
|
||
The patch that was checked into the trunk fixes this problem in the Privacy
panel. (I tested by switching to High Contrast mode, left Alt + left Shift +
PrintScreen.) Was it also checked into the trunk?
Keywords: fixed-aviary1.0
Comment 23•20 years ago
|
||
You say privacy panel... the other part was checked in, right? The added
WindowText property to fix the side panel?
Comment 24•20 years ago
|
||
*** Bug 272793 has been marked as a duplicate of this bug. ***
Comment 25•20 years ago
|
||
Fixed on the aviary branch 2004-10-12 20:05.
Fixed on trunk by the branch landing.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Attachment #154281 -
Attachment is obsolete: true
Attachment #154281 -
Flags: review?(vladimir)
Comment 26•18 years ago
|
||
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs,
filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → preferences
You need to log in
before you can comment on or make changes to this bug.
Description
•