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 781214 - define entry-tag in Adwaita.
define entry-tag in Adwaita.
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
3.90.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks: 781671
 
 
Reported: 2017-04-12 12:18 UTC by Jakub Steiner
Modified: 2017-05-11 13:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
style entry-tags (991 bytes, patch)
2017-04-13 17:39 UTC, Jakub Steiner
none Details | Review
style entry-tags (834 bytes, patch)
2017-04-20 08:37 UTC, Jakub Steiner
committed Details | Review
*scss only* patch (2.15 KB, patch)
2017-05-11 12:43 UTC, Lapo Calamandrei
none Details | Review

Description Jakub Steiner 2017-04-12 12:18:12 UTC
When searching, scope is shown as tags in the search entry. This is currently style in individual applications as documents-entry-tag and photos-entry-tag for example.

Would be nice to define it in adwaita as entry-tag.
Comment 1 Jakub Steiner 2017-04-13 17:39:19 UTC
Created attachment 349832 [details] [review]
style entry-tags
Comment 2 Lapo Calamandrei 2017-04-15 13:38:53 UTC
Jakub, not sure about the background color, darkening $bg_color 20% on a the dark variant won't probably work that nicelly ($base_color is darker then $bg_color there), also to reset the button we have a nice mixin (button(undecorated) or something IIRC), please use that one.
Comment 3 Lapo Calamandrei 2017-04-15 13:40:54 UTC
also, backdrop state?
Comment 4 Jakub Steiner 2017-04-20 08:37:10 UTC
Created attachment 350103 [details] [review]
style entry-tags
Comment 5 Debarshi Ray 2017-04-20 16:14:24 UTC
Jimmac is OK with having styling for libgd widgets in gtk+, so this doesn't depend on bug 756353.
Comment 6 Cosimo Cecchi 2017-04-20 17:33:55 UTC
Review of attachment 350103 [details] [review]:

The code looks good to me. However, we will need to change the style class in libgd for this to work.
When doing that, libgd currently also adds a CSS provider with the default theming for the entry tag, which should be removed as part of this.
Comment 7 Matthias Clasen 2017-05-08 21:55:18 UTC
Attachment 350103 [details] pushed as dee666f - style entry-tags
Comment 8 Debarshi Ray 2017-05-09 11:14:27 UTC
Review of attachment 350103 [details] [review]:

Is this all that is to it?

I can't get gnome-documents/photos.git master to pick up the new entry-tag styling. I did install sassc and rebuilt gtk-3-22 and my config.log says:
configure:29158: checking for sassc
configure:29176: found /usr/bin/sassc
configure:29188: result: /usr/bin/sass

Is this a matter of having an entry-tag CSS style class, as opposed to just a CSS node? Nodes are not available to out-of-tree widgets.
Comment 9 Lapo Calamandrei 2017-05-10 14:07:10 UTC
The entry-tag looks pretty ugly in context, reopening. I'll work on a better style, but the entry itself has some issue, e.g. it doesn't seem to respond to :dir(rtl) for example and I'd like to know if the entry tag styling is just for the libgd implementation, if it is I can fix some specific issue, like spacing and placement with some kind of gross hack in adwaita (read negative margins to componsate entry padding for a better placement)
Comment 10 Lapo Calamandrei 2017-05-11 12:43:37 UTC
Created attachment 351622 [details] [review]
*scss only* patch

There's my proposed styling, I used a blue on the bright variant, similar to what gnome-documents was shipping and a gray for the dark variant, which looks more usable to me.
There are some quirks with the entry-tag implementation though, the tag close button doesn't get the right style until is hovered and the active state on that button remains stuck in certain cases.
Comment 11 Lapo Calamandrei 2017-05-11 13:28:19 UTC
Pushed to master and 3.22, closing, please reopen in case of issues (with the style, otherwise blame Debarashi :-))