After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 757164 - Update theme for GTK+ 3.19
Update theme for GTK+ 3.19
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
unspecified
Other All
: High blocker
: ---
Assigned To: Michael Catanzaro
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-10-27 02:26 UTC by Michael Catanzaro
Modified: 2016-01-13 04:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Update for removal of GTK_STYLE_CLASS_ENTRY (2.41 KB, patch)
2015-10-27 02:26 UTC, Michael Catanzaro
none Details | Review
Regenerate CSS with current GTK+ master (8.51 KB, patch)
2015-12-20 19:38 UTC, Michael Catanzaro
committed Details | Review
Remove theme for old download box (3.35 KB, patch)
2015-12-20 19:38 UTC, Michael Catanzaro
committed Details | Review
Update for removal of GTK_STYLE_CLASS_ENTRY (2.49 KB, patch)
2015-12-20 19:38 UTC, Michael Catanzaro
committed Details | Review
Update the theme for CSS nodes (7.07 KB, patch)
2015-12-20 19:38 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2015-10-27 02:26:37 UTC
This does NOT fix the SCSS file; I need Lapo to help with that, since I can't figure out how to compile the SCSS. (Also note we have unused SCSS for the downloads bar that was removed.)
Comment 1 Michael Catanzaro 2015-10-27 02:26:40 UTC
Created attachment 314176 [details] [review]
Update for removal of GTK_STYLE_CLASS_ENTRY
Comment 2 Michael Catanzaro 2015-10-27 02:27:45 UTC
CC: Lapo (help please :)

Note: Don't push this until the version bump to GTK+ 3.19.1 occurs
Comment 3 Lapo Calamandrei 2015-10-27 11:31:12 UTC
Michael, GTK+ CSS is pretty much in flux still, let's wait for the the button styleclass to change as we'll before looking into this, OK?
Comment 4 Michael Catanzaro 2015-12-20 19:38:15 UTC
Created attachment 317700 [details] [review]
Regenerate CSS with current GTK+ master
Comment 5 Michael Catanzaro 2015-12-20 19:38:21 UTC
Created attachment 317701 [details] [review]
Remove theme for old download box

It's gone
Comment 6 Michael Catanzaro 2015-12-20 19:38:25 UTC
Created attachment 317702 [details] [review]
Update for removal of GTK_STYLE_CLASS_ENTRY
Comment 7 Michael Catanzaro 2015-12-20 19:38:55 UTC
Created attachment 317703 [details] [review]
Update the theme for CSS nodes

button and entry are now CSS names, not style classes.

I probably missed something, or screwed this up, so a review would be awesome (Lapo ;)
Comment 8 Michael Catanzaro 2015-12-20 19:41:36 UTC
By the way, see https://bugs.webkit.org/show_bug.cgi?id=150550 for the corresponding WebKit bug, if you want a usable web browser or want to help me figure out how to draw progress bars again. The patch in that bug fixes stuff inside the web view (like Bugzilla forms and selections and the scrollbar).
Comment 9 Michael Catanzaro 2015-12-20 19:45:06 UTC
Arnaud, I think you're probably qualified to review this too? You certainly understand CSS better than I do....
Comment 10 Lapo Calamandrei 2015-12-21 09:44:31 UTC
Review of attachment 317701 [details] [review]:

Looks good
Comment 11 Michael Catanzaro 2015-12-21 13:22:07 UTC
Attachment 317700 [details] pushed as 47b95d8 - Regenerate CSS with current GTK+ master
Attachment 317701 [details] pushed as aa5f69f - Remove theme for old download box
Attachment 317702 [details] pushed as f74683a - Update for removal of GTK_STYLE_CLASS_ENTRY
Attachment 317703 [details] pushed as 19ff1cd - Update the theme for CSS nodes
Comment 12 Michael Catanzaro 2015-12-21 13:59:16 UTC
Incognito mode title bar icon is broken:

** (epiphany:12502): WARNING **: CSS error in section beginning line 29 at offset 20:
 Not a valid image

The line in question:

  background-image: -gtk-icontheme("user-not-tracked-symbolic"), headerbar_gradient(#cbd2d9);