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 763623 - CSS runtime warning
CSS runtime warning
Status: RESOLVED FIXED
Product: polari
Classification: Applications
Component: general
3.19.x
Other Linux
: Normal normal
: ---
Assigned To: Polari maintainers
Polari maintainers
Depends on:
Blocks:
 
 
Reported: 2016-03-14 15:08 UTC by Claudio André
Modified: 2016-03-16 17:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (1.08 KB, patch)
2016-03-14 15:10 UTC, Claudio André
committed Details | Review

Description Claudio André 2016-03-14 15:08:06 UTC
There is one run-time warning.

(org.gnome.Polari:22033): Gtk-WARNING **: Theme parsing error: application.css:18:30: The style property GtkWidget:focus-padding is deprecated and shouldn't be used anymore. It will be removed in a future version
---

To fix this we should remove 'GtkWidget-focus-padding' and use the CSS standard 'padding' property instead.


I
Comment 1 Claudio André 2016-03-14 15:10:29 UTC
Created attachment 323878 [details] [review]
Proposed patch
Comment 2 Florian Müllner 2016-03-14 20:31:54 UTC
Review of attachment 323878 [details] [review]:

> To fix this we should remove 'GtkWidget-focus-padding' and use the CSS
> standard 'padding' property instead.

No, that replacement is not correct. The former used to affect the focus indication (that dotted rectangle when tab'ing around the interface), while the standard padding affects the content box.
Comment 3 Claudio André 2016-03-15 02:14:38 UTC
This is the recommended solution seen in GNOME docs
- https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--s-focus-padding

And, it seems to work as expected. Well, maybe I failed to see the differences.
Comment 4 Florian Müllner 2016-03-15 08:46:48 UTC
(In reply to Claudio André from comment #3)
> This is the recommended solution seen in GNOME docs
> -
> https://developer.gnome.org/gtk3/stable/GtkWidget.html#GtkWidget--s-focus-
> padding

Yet when updating Adwaita itself[0], the property was not replaced ...

[0] https://git.gnome.org/browse/gtk+/commit?id=e24c248013b6d6698
Comment 5 Matthias Clasen 2016-03-15 13:04:50 UTC
The correct replacement would be outline-offset, I guess
Comment 6 Claudio André 2016-03-15 18:14:47 UTC
I'm leaving this to you guys. The best I can think is something like this:

.polari-user-list row {
    padding: 0px
    outline: 1px dotted gray;
    outline-offset: 0px;  
}

BTW: default value of outline-offset is zero.
Comment 7 Florian Müllner 2016-03-16 17:47:05 UTC
To be honest, I don't quite remember why the property is there in the first place (git-blame points to me, but I suspect that the style part was done by Jakub). Let's just remove it to get rid of the warning, and let the designers come up with fixes for anything they think needs fixing ...