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 704583 - shell-app: Fade the app icon on the left in RTL layouts
shell-app: Fade the app icon on the left in RTL layouts
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-07-19 19:45 UTC by Jasper St. Pierre (not reading bugmail)
Modified: 2013-07-22 11:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
shell-app: Fade the app icon on the left in RTL layouts (5.35 KB, patch)
2013-07-19 19:45 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description Jasper St. Pierre (not reading bugmail) 2013-07-19 19:45:12 UTC
See patch for rationale.
Comment 1 Jasper St. Pierre (not reading bugmail) 2013-07-19 19:45:14 UTC
Created attachment 249657 [details] [review]
shell-app: Fade the app icon on the left in RTL layouts

The point of fading the icon is to make the text displayed over the
icon more legible. In RTL layouts, the text is displayed on the left
of the icon, so fading the right-hand-side of the icon doesn't work
well.
Comment 2 Giovanni Campagna 2013-07-22 11:04:59 UTC
Review of attachment 249657 [details] [review]:

Just a style nit

::: src/shell-app.c
@@ +302,3 @@
           guchar *pixel = &pixels[j * rowstride + i * n_channels];
+          float fade = ((float) i - fade_start) / (fade_end - fade_start);
+          /* fade at the right side for LTR, left side for RTL */

Place this comment above the first if (data->direction)
Comment 3 Jasper St. Pierre (not reading bugmail) 2013-07-22 11:11:28 UTC
Attachment 249657 [details] pushed as 1cc5bb5 - shell-app: Fade the app icon on the left in RTL layouts