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 751787 - Images of refresh button and logo are pixelated on scaled outputs
Images of refresh button and logo are pixelated on scaled outputs
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: .General
3.16.x
Other Linux
: Normal enhancement
: ---
Assigned To: gtk-bugs
gtk-bugs
http://linkmauve.fr/files/epiphany-hi...
Depends on:
Blocks:
 
 
Reported: 2015-07-01 15:19 UTC by bugs
Modified: 2015-09-30 22:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
hidpi working fine (125.96 KB, image/png)
2015-07-01 17:55 UTC, Jakub Steiner
Details

Description bugs 2015-07-01 15:19:01 UTC
On scaled outputs (HiDPI), those two images are quite ugly compared to the rest of the buttons.

Providing a vector version of them would fix this issue for arbitrary scaling properties.
Comment 1 Michael Catanzaro 2015-07-01 16:21:48 UTC
The refresh button is 'view-refresh-symbolic' (it is actually an svg, so not sure how it's being pixellated). The icon used for the logo is 'web-browser'.
Comment 2 bugs 2015-07-01 16:32:40 UTC
I tested on Weston, with everything stock from ArchLinux (except webkit2gtk), if that helps reproducing.

Thanks for changing the product to the one it belongs to. :)
Comment 3 Jakub Steiner 2015-07-01 17:24:13 UTC
There is nothing specific about view-refresh-symbolic, it's the same scalable vector SVG as all the other icons. Same for web-browser-symbolic. Can you provide a screenshot?
Comment 4 bugs 2015-07-01 17:40:08 UTC
I added a screenshot in the URL field, and noticed the stop button appear non-vectorial as well.
Comment 5 Jakub Steiner 2015-07-01 17:55:34 UTC
Created attachment 306550 [details]
hidpi working fine
Comment 6 Jakub Steiner 2015-07-01 17:56:57 UTC
I'm afraid I can't reproduce the issue locally. In any case it's hardly an issue with the icon theme itself, throwing the hot potato back to epiphany :)
Comment 7 Michael Catanzaro 2015-07-01 18:16:16 UTC
Is it possible that the icon becomes blurry after switching from Reload to Stop and back? If so, we can assign this to GTK+. Otherwise, I have no clue what to do with this bug.
Comment 8 bugs 2015-07-01 18:42:04 UTC
It doesn’t seem so, when I load Epiphany on about:blank the stop button looks pixelated too.
Comment 9 bugs 2015-07-01 18:44:54 UTC
I can actually see view-refresh-symbolic.svg being loaded, in strace, so the issue wouldn’t be that it isn’t vectorial but that it gets rendered at the wrong size, or something.

The only PNG files loaded by Epiphany are web-browser.png at various sizes: 22, 16, 24, 32, 256, 48, then 16 again.
Comment 10 David Gumberg 2015-07-01 18:48:43 UTC
Can you post a screenshot?
Comment 11 bugs 2015-07-01 18:57:07 UTC
I did already, in the URL field: http://linkmauve.fr/files/epiphany-hidpi.png
Comment 12 Michael Catanzaro 2015-07-02 01:23:52 UTC
Hot potato -> GTK+, since I have no clue what else to do with this bug. Strange that it works for Jakub but not you. (Presumably Weston does not have anything to do with that... but maybe it does.)
Comment 13 Matthias Clasen 2015-07-02 05:07:22 UTC
works fine here as well
Comment 14 bugs 2015-09-27 14:32:16 UTC
I just managed to reproduce on a 3.18 stack, the refresh icon is good if only my DP1 output is enabled in Weston’s config file with scale=2, and bad if my LVDS1 output is also enabled with scale=1.  It is also good when my LVDS1 output is enabled with scale=2.
Comment 15 Matthias Clasen 2015-09-28 11:58:35 UTC
And this is a tool button ? How is it constructed ?
Comment 16 Matthias Clasen 2015-09-28 13:00:21 UTC
ignore the tool button question - I've looked at the screenshot now. Still, knowing how the buttons are constructed would be helpful.
Comment 17 Michael Catanzaro 2015-09-28 13:28:02 UTC
In ephy-toolbar.c, ephy_toolbar_constructed():

  /* Reload/Stop */
  button = gtk_button_new ();
  /* FIXME: apparently we need an image inside the button for the action
   * icon to appear. */
  gtk_button_set_image (GTK_BUTTON (button), gtk_image_new ());
  gtk_widget_set_valign (button, GTK_ALIGN_CENTER);
  action = gtk_action_group_get_action (action_group, "ViewCombinedStopReload");
  gtk_activatable_set_related_action (GTK_ACTIVATABLE (button),
                                      action);
  gtk_header_bar_pack_start (GTK_HEADER_BAR (toolbar), button);
Comment 18 Matthias Clasen 2015-09-30 22:44:31 UTC
Fixed the menu button. Can't reproduce the other button problem.