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 726907 - More fixes for HiDpi support
More fixes for HiDpi support
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 728382 728393 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-03-23 04:08 UTC by Cosimo Cecchi
Modified: 2014-04-17 16:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
texture-cache: use scale factor to load sliced image (6.09 KB, patch)
2014-03-23 04:08 UTC, Cosimo Cecchi
committed Details | Review
texture-cache: always call gdk_pixbuf_loader_set_size (2.12 KB, patch)
2014-03-23 04:08 UTC, Cosimo Cecchi
committed Details | Review
border-image: add support for scale factor (3.87 KB, patch)
2014-03-23 04:08 UTC, Cosimo Cecchi
committed Details | Review
texture-cache: use scale factor to load background and borders (11.08 KB, patch)
2014-03-23 04:08 UTC, Cosimo Cecchi
committed Details | Review
texture-cache: use scale factor for load_uri_async() (5.21 KB, patch)
2014-03-23 04:08 UTC, Cosimo Cecchi
committed Details | Review

Description Cosimo Cecchi 2014-03-23 04:08:31 UTC
This patchset adds some missing pieces to the HiDpi support in gnome-shell. Namely, now background images, border images and other textures loaded by StTextureCache will all be scaled by the display scale factor when needed.
That fixes the appearance of switches, close buttons, borders and highlights for panel icons, and more.
Comment 1 Cosimo Cecchi 2014-03-23 04:08:34 UTC
Created attachment 272674 [details] [review]
texture-cache: use scale factor to load sliced image

We need to use a GdkPixbufLoader instead of the straightforward
gdk_pixbuf_new_from_file(), since we want to load the image already
scaled if possible - e.g. if it's an SVG file.
Comment 2 Cosimo Cecchi 2014-03-23 04:08:44 UTC
Created attachment 272675 [details] [review]
texture-cache: always call gdk_pixbuf_loader_set_size

Just use the regular image size if we don't want to scale it. This is a
refactor in preparation for the next commit.
Comment 3 Cosimo Cecchi 2014-03-23 04:08:48 UTC
Created attachment 272676 [details] [review]
border-image: add support for scale factor

In a later commit we'll add support for rendering borders and
backgrounds with scale factor.
Comment 4 Cosimo Cecchi 2014-03-23 04:08:52 UTC
Created attachment 272677 [details] [review]
texture-cache: use scale factor to load background and borders
Comment 5 Cosimo Cecchi 2014-03-23 04:08:55 UTC
Created attachment 272678 [details] [review]
texture-cache: use scale factor for load_uri_async()
Comment 6 drago01 2014-03-23 16:40:32 UTC
Review of attachment 272674 [details] [review]:

OK.
Comment 7 drago01 2014-03-23 16:43:47 UTC
Review of attachment 272675 [details] [review]:

OK.
Comment 8 drago01 2014-03-23 16:44:24 UTC
Review of attachment 272676 [details] [review]:

OK.
Comment 9 drago01 2014-03-23 16:44:25 UTC
Review of attachment 272676 [details] [review]:

OK.
Comment 10 drago01 2014-03-23 16:46:02 UTC
Review of attachment 272677 [details] [review]:

OK.
Comment 11 drago01 2014-03-23 16:49:46 UTC
Review of attachment 272678 [details] [review]:

Looks good as well ... marked them all ac-after-freeze because we are frozen. They can go into 3.12.1 though.
Comment 12 drago01 2014-03-23 16:51:21 UTC
Review of attachment 272678 [details] [review]:

Looks good as well ... marked them all ac-after-freeze because we are frozen. They can go into 3.12.1 though.
Comment 13 Cosimo Cecchi 2014-03-28 17:53:27 UTC
Attachment 272674 [details] pushed as 74d9b6c - texture-cache: use scale factor to load sliced image
Attachment 272675 [details] pushed as 90e52d7 - texture-cache: always call gdk_pixbuf_loader_set_size
Attachment 272676 [details] pushed as a22fdea - border-image: add support for scale factor
Attachment 272677 [details] pushed as 9ecf466 - texture-cache: use scale factor to load background and borders
Attachment 272678 [details] pushed as 9c88fec - texture-cache: use scale factor for load_uri_async()
Comment 14 Cosimo Cecchi 2014-04-16 22:52:02 UTC
*** Bug 728382 has been marked as a duplicate of this bug. ***
Comment 15 Cosimo Cecchi 2014-04-17 16:18:54 UTC
*** Bug 728393 has been marked as a duplicate of this bug. ***