Closed
Bug 1198063
Opened 9 years ago
Closed 9 years ago
[GTK3] Info bar text is really hard to read (light-gray-on-yellow) in default Ubuntu theme, even after bug 1187203
Categories
(Core :: Widget: Gtk, defect)
Core
Widget: Gtk
Tracking
()
RESOLVED
INVALID
Tracking | Status | |
---|---|---|
firefox41 | --- | unaffected |
firefox42 | --- | affected |
firefox43 | --- | affected |
People
(Reporter: dholbert, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(7 files, 1 obsolete file)
Bug 1187203 was filed on white-on-white infobar text.
Its patch improved things a bit, but on my system at least (running Ubuntu 15.04 with gnome-shell & default "Ambiance" theme), it's still noticeably hard to read.
See attached screenshot, which shows an "[Add-on] might be making Nightly run slowly" in one Firefox window, and the top of an RSS feed* in a different Firefox window. Both have light-gray text on a yellow background, and look unnecessarily hard to read. The text looks like a disabled formcontrol, or something like that -- like it's there but not meant to be read.
* NOTE: To test RSS feeds [just one way of easily visualizing how this looks], you have to turn off e10s, since RSS rendering apparently doesn't work yet in e10s.
I'm using today's Nightly 43.0a1 (2015-08-24)
Reporter | ||
Comment 1•9 years ago
|
||
For comparison, Firefox 40 (with GTK2) renders these sections with white text on a black background.
So, there's sharp contrast, and it's much easier to read (and much less "disabled"/"you-don't-want-to-read-me" looking) as compared to current release.
Reporter | ||
Updated•9 years ago
|
Attachment #8652084 -
Attachment description: screenshot of post-bug-1187203 rendering → screenshot of post-bug-1187203 rendering, showing light-gray-on-yellow
Reporter | ||
Comment 2•9 years ago
|
||
Here's a simple testcase, using the -moz-gtk-info-bar appearance and text-color that was added in bug 1187203.
In current Nightly, this renders like my first light-gray-on-yellow screenshot here.
Reporter | ||
Comment 3•9 years ago
|
||
Andrew, I'm not sure where the colors here come from -- is this another case where my theme is being silly? Or are these colors controlled by Gecko? (in which case, could we improve the contrast?)
Flags: needinfo?(acomminos)
Comment 4•9 years ago
|
||
Ack, it's unfortunately quite likely to be your theme- the way we present the notification bars should styled identically to a GtkInfoBar with a GtkLabel in its content area.
Can you let me know if the attached program has difficult to read text as well? Thanks!
Flags: needinfo?(acomminos)
Reporter | ||
Comment 5•9 years ago
|
||
Reporter | ||
Comment 6•9 years ago
|
||
Yeah, so Ambiance has:
gtk-main.css:@define-color info_fg_color rgb (181, 171, 156);
gtk-main.css:@define-color info_bg_color rgb (252, 252, 189);
..which is light-gray-on-yellow. The yellow ends up being used in a gradient expression (which we dutifully render now), and it ends up pretty unreadable, as shown in the screenshot of Andrew's example GTK program.
So, this is a bug in Ambiance, basically. (And a more legitimate bug than the one I spun off of bug 1187203 -- in that case, Ambiance was just expecting that its background gradients would be rendered. Here, we are rendering them, and it's still unreadable.)
So I guess this is INVALID as a Firefox bug, unless we decide we want to use something other than a GTK InfoBar here (for our Firefox infobars) -- that would work around this on our end. (e.g. Before bug 1187203, it looked like we were trying [and not quite succeeding] to use GTK tooltip styling for our info-bars. If we decide to go back to using a tooltip under the hood, then that would get around this...)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
Updated•9 years ago
|
status-firefox41:
--- → unaffected
status-firefox42:
--- → affected
Comment 8•9 years ago
|
||
On Linux Mint the colors are white text on gray background.
Is someone filing a bug in the relevant external project?
Reporter | ||
Comment 9•9 years ago
|
||
(In reply to Christian Ascheberg from comment #8)
> On Linux Mint the colors are white text on gray background.
Yikes, that is horrible.
> Is someone filing a bug in the relevant external project?
Could you please do so (for Linux Mint)? I don't have Linux Mint and I don't know where to file bugs for its themes. (You may not either, but you're in a better position than I am to report bugs to them, since you're a user.)
Answering your question -- in the case of my screenshots here, I haven't bothered to report an upstream bug because:
(1) The text is actually readable, though just a bit annoying. So, not a deal-breaker.
(2) The theme is behaving as-designed, as shown by acomminos' trivial demo-program. (This isn't a Firefox-specific issue in any way.)
(3) I doubt an Ubuntu bug will get traction. I filed an upstream bug* for the more-severe & easier-to-fix bug 1478173, with a patch, and it still hasn't gotten any comments 2 months later. (Just one comment from a 3rd-party theme developer that I reached out to separately via email). So, given that this bug is less-bad (on Ubuntu's theme), and possibly working how the theme designer intended, and it's not as obvious how to fix (gotta pick a new color, what color?), I have doubts that an upstream bug would have any effect.
* https://bugs.launchpad.net/ubuntu/+source/ubuntu-themes/+bug/1478173
Comment 10•9 years ago
|
||
Hmm I compiled the GtkInfoBar example program and there the text is black. So could it still be a bug in Firefox?
Attachment #8662972 -
Attachment is obsolete: true
Reporter | ||
Comment 11•9 years ago
|
||
That sounds like it could be a bug, then. Could you file a new bug on this Mint-specific issue [which it seems could be a Firefox bug], with that screenshot, and CC acomminos and karlt?
Reporter | ||
Updated•9 years ago
|
Comment 12•8 years ago
|
||
This presents the colors we would get from Ambiance on Ubuntu 14.04 if we were to consider using different message types, if appropriate.
However, the only one which has helpful contrast is the error bar, and IMO that is too striking for most notifications.
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•