Closed
Bug 982964
Opened 11 years ago
Closed 11 years ago
[gtk3] Combo boxes have white text
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
RESOLVED
FIXED
mozilla31
Tracking | Status | |
---|---|---|
firefox31 | --- | fixed |
People
(Reporter: hub, Assigned: stransky)
References
Details
Attachments
(3 files, 1 obsolete file)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
karlt
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
karlt
:
review+
|
Details | Diff | Splinter Review |
I built Firefox with Gtk3 support on Fedora 20. I use Gnome3 with the default theme.
The combo box have white text which make them hard if not impossible to read.
See screenshot.
Assignee | ||
Comment 1•11 years ago
|
||
Which theme do you run and which Firefox version?
Flags: needinfo?(hub)
Reporter | ||
Comment 3•11 years ago
|
||
Default theme on Fedora 20 and 175595:210213ae7c1c from mozilla-inbound (build from source)
Assignee | ||
Comment 4•11 years ago
|
||
We use a wrong color for the button text in the hover event. It should be the label color, not button color.
Attachment #8398448 -
Flags: review?(karlt)
Reporter | ||
Comment 5•11 years ago
|
||
It fixes the hover, indeed - which I also observed but didn't really report. But still not the combo boxes.
It still look like that on the screenshot.
Updated•11 years ago
|
Attachment #8398448 -
Flags: review?(karlt) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Whiteboard: [leave open for remaining patches]
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Hubert Figuiere [:hub] from comment #5)
> It fixes the hover, indeed - which I also observed but didn't really report.
> But still not the combo boxes.
I see. It's because comboboxes on the web page uses a different color than comboboxes in Chrome (Menu/Preferences).
Assignee | ||
Comment 7•11 years ago
|
||
I's because this mechanism:
gtk_container_add(GTK_CONTAINER(combobox), comboboxLabel);
no longer works. The combobox does not have set "has-entry" property and the label is not considered as a combobox entry. We need to create a combobox with entry and get the entry from combobox.
Assignee | ||
Comment 8•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → stransky
Reporter | ||
Comment 9•11 years ago
|
||
Oh yeah this is so much better. Thanks !!!
Assignee | ||
Comment 10•11 years ago
|
||
Attachment #8399438 -
Attachment is obsolete: true
Attachment #8399816 -
Flags: review?(karlt)
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Whiteboard: [leave open for remaining patches]
Comment 11•11 years ago
|
||
Comment on attachment 8399816 [details] [diff] [review]
combobox text patch
(Label was probably the wrong type of widget for GTK2 also.)
Attachment #8399816 -
Flags: review?(karlt) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 12•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/69ea6095a7aa
https://hg.mozilla.org/integration/mozilla-inbound/rev/a288dcaadccc
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/69ea6095a7aa
https://hg.mozilla.org/mozilla-central/rev/a288dcaadccc
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Reporter | ||
Comment 15•10 years ago
|
||
I can't I stopped building for gtk3 because of another blocker....
Flags: needinfo?(hub)
Comment 16•10 years ago
|
||
Anyone can confirm by using the builds from the elm project branch.
Comment 17•10 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #16)
> Anyone can confirm by using the builds from the elm project branch.
Thanks.
status-firefox-esr31:
--- → fixed
Keywords: verifyme
Comment 18•10 years ago
|
||
Sorry, wrong flag.
status-firefox31:
--- → fixed
status-firefox-esr31:
fixed → ---
Comment 19•10 years ago
|
||
We only found elm Firefox builds for Firefox 33 linux x64. Can someone provide us with a Firefox 31 elm build in order to verify the bug?
Flags: needinfo?(mh+mozilla)
Comment 20•10 years ago
|
||
Gtk+3 is a work in progress currently, and there were many problems in 31 with its. It may not even build. The first version that kind of works in 32, but really, I think it's not worth worrying more than 33 at this point.
Flags: needinfo?(mh+mozilla)
You need to log in
before you can comment on or make changes to this bug.
Description
•