Bug 69280

Summary:	UI: Comments toggle button should be on the left side of the ruler when locale set to RTL language
Product:	LibreOffice	Reporter:	Maxim Monastirsky <momonasmon>
Component:	Writer	Assignee:	Not Assigned <libreoffice-bugs>
Status:	VERIFIED FIXED
Severity:	minor	CC:	kaplanlior, philipz85
Priority:	medium
Version:	4.1.1.2 release
Hardware:	Other
OS:	Linux (All)
Whiteboard:	target:4.2.0
Crash report or crash signature:		Regression By:
Bug Depends on:
Bug Blocks:	43808, 106179
Attachments:	screenshot (Fedora 19) screenshot with RTL paragraph (Fedora 19)

Description Maxim Monastirsky 2013-09-12 17:00:56 UTC

Created attachment 85729 [details]
screenshot (Fedora 19)

Steps to reproduce:
1) Set locale to some RTL language (like Arabic or Hebrew) (through Tools->Options...->Language settings->Languages->locale setting)
2) Open Writer and add comment (Insert->Comment)
3) As you can see the comment appears on the left side of the page, while the toggle button (which was added due to bug 38246) appears on the right side of the ruler, not above the comments.

Tested with LO 4.1.1.2 under both Ubuntu 13.10 & Fedora 19.

Comment 1 Maxim Monastirsky 2013-09-15 16:10:45 UTC

Created attachment 85871 [details]
screenshot with RTL paragraph (Fedora 19)

Also please note that when paragraph's direction is RTL, you don't see the tab
icon on the top ruler (probably because comments toggle button covering it, or pushing it too much to right). So it's missing functionality, not just cosmetic issue.

Comment 2 Lior Kaplan 2013-09-15 21:08:21 UTC

Bug confirmed (4.1.1.2 on Debian 64bit). The tab icon is available if you zoom out a little (meaning it's there, but covered as you thought).

Comment 3 Commit Notification 2013-09-18 10:47:22 UTC

Abdulaziz A Alayed committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d86a72b64f208fe55f302d9fd8a7f578a4275bb8

fdo#69280 fixing comments toggle button position when local



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

Comment 4 Lior Kaplan 2013-09-20 18:43:37 UTC

Fix verified on a build from master.