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 702760 - Use in rtl-symbolic instead in symbol in the opposite direction
Use in rtl-symbolic instead in symbol in the opposite direction
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-06-20 17:17 UTC by Yosef Or Boczko
Modified: 2013-08-12 11:14 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Use in rtl-symbolic instead in symbol in the opposite direction (965 bytes, patch)
2013-06-20 17:17 UTC, Yosef Or Boczko
none Details | Review
Use in rtl-symbolic instead in symbol in the opposite direction (1.00 KB, patch)
2013-07-01 07:24 UTC, Vadim Rutkovsky
committed Details | Review
Screenshot - before (291.90 KB, image/png)
2013-08-12 11:12 UTC, Yosef Or Boczko
  Details
Screenshot - after (288.13 KB, image/png)
2013-08-12 11:12 UTC, Yosef Or Boczko
  Details

Description Yosef Or Boczko 2013-06-20 17:17:37 UTC
Created attachment 247363 [details] [review]
Use in rtl-symbolic instead in symbol in the opposite direction

See patch.
Comment 1 Vadim Rutkovsky 2013-06-21 17:49:29 UTC
Looks reasonable, is there any way to test this (or please give links to similar patches)
Comment 2 Yosef Or Boczko 2013-06-22 17:43:15 UTC
Of course:
In goobox:
https://bugzilla.gnome.org/show_bug.cgi?id=702661

In gnome-clocks:
https://bugzilla.gnome.org/show_bug.cgi?id=702589
Comment 3 Yosef Or Boczko 2013-07-01 07:09:44 UTC
You push the patch?
Comment 4 Vadim Rutkovsky 2013-07-01 07:24:22 UTC
Thanks, pushed, please verify

The following fix has been pushed:
9945025 Use in rtl-symbolic instead in symbol in the opposite direction
Comment 5 Vadim Rutkovsky 2013-07-01 07:24:26 UTC
Created attachment 248114 [details] [review]
Use in rtl-symbolic instead in symbol in the opposite direction
Comment 6 Yosef Or Boczko 2013-07-01 07:35:49 UTC
Requires many fixes:

media-playback-start-symbolic in LTR,
media-playback-start-rtl-symbolic in RTL:
https://git.gnome.org/browse/gnome-music/tree/data/PlayerToolbar.ui#n389
https://git.gnome.org/browse/gnome-music/tree/src/view.js#n44
https://git.gnome.org/browse/gnome-music/tree/src/widgets.js#n39

media-skip-forward-symbolic in LTR,
media-skip-forward-rtl-symbolic in RTL:
https://git.gnome.org/browse/gnome-music/tree/data/PlayerToolbar.ui#n30

media-skip-backward-symbolic in LTR,
media-skip-backward-rtl-symbolic in RTL:
https://git.gnome.org/browse/gnome-music/tree/data/PlayerToolbar.ui#n396

See the bug in totem:
https://bugzilla.gnome.org/show_bug.cgi?id=702631

(I'm not a programmer in JavaScript)
Comment 7 Yosef Or Boczko 2013-07-01 07:37:26 UTC
(In reply to comment #5)
> Created an attachment (id=248114) [details] [review]
> Use in rtl-symbolic instead in symbol in the opposite direction

verified.
Comment 8 Vadim Rutkovsky 2013-07-03 11:20:51 UTC
(In reply to comment #6)
> Requires many fixes:
Reopening
Comment 9 Yosef Or Boczko 2013-08-12 11:12:09 UTC
Created attachment 251335 [details]
Screenshot - before
Comment 10 Yosef Or Boczko 2013-08-12 11:12:34 UTC
Created attachment 251336 [details]
Screenshot - after
Comment 11 Yosef Or Boczko 2013-08-12 11:14:06 UTC
Thanks for Arnel A. Borja for his work - for this fix:
https://git.gnome.org/browse/gnome-music/commit/?id=cf8c8c80ce400220b23f932198340fa3f1e6e307