Created attachment 122082 [details] format cells dialog If CTL isnt enabled and you open up the format cells dialog in Calc, you dont have access to the text direction feature. This can be a problem with a user who receives a document where cells where set to RTL or LTR, but isnt able to find any means of changing this setting if he doesnt have CTL enabled. I was having this same problem when testing attachment 115079 [details] from bug 58122 and I didnt have CTL enabled, but was sure that it had to be found in the format cell dialog. I'm hoping this is an easy fix that can be solved simply by modifying the dialog's UI file, but there must be some coding involved which causes it to show or hide. Version: 5.2.0.0.alpha0+ Build ID: d95d9d7f908419f397941ef60ac6ced3261c9b87 CPU Threads: 2; OS Version: Linux 4.2; UI Render: default; TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2016-01-19_00:40:21 Locale: en-US (en_US.UTF-8)
Those features are hidden because they are confusing for LTR-only users. Also it is unclear why such a user should work with RTL spreadsheets.
I seem to be confused that LTR users would find the text direction feature confusing when LTR users can set a cell contents to align right.
So this seems to be a simple fix so lets give it a go. https://gerrit.libreoffice.org/22673 @Maxim, @Samuel, @Bubli: Would this patch have any negative affects.
@Cor, @Stuart, @Adolfo: Some have given their opposition to this and i was wondering what your opinion is? As someone who doesnt enable CTL as it isnt enabled by default, i do receive documents in arabic and having the ability to atleast switch in the cell or paragraph dialog the text direction to RTL is a benefit.
I see no problem in adding this option. Some users may wonder themselves when they first see this, but that won't do much harm. After a simple experiment (for those few users that do so with unknown stuff) it should be clear to them.
(In reply to Yousuf (Jay) Philips from comment #4) If the setting can be simply enabled without major refactoring of the CTL language modes--or even forcing user to set a default CTL, it probably ought to be. Seems like it would simplify some of the XCU for .ui navigation.
(In reply to V Stuart Foote from comment #6) > If the setting can be simply enabled without major refactoring of the CTL > language modes--or even forcing user to set a default CTL, it probably ought > to be. Seems like it would simplify some of the XCU for .ui navigation. Yes the patch simply removes the if ( CTL is enabled ) { show text direction } from the code to always show it.
While having access to the text direction under alignment is a good option and generates awareness for the LTR/RTL difference, I think the font tab is inferior and unfamiliar with CTL enabled. But since Jay's patch affects the alignment tab only I'd say "ship it"!
Yousuf Philips committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=93e607517342cc3fa672957535570d1790058807 tdf#97251 Always show the text direction alignment control It will be available in 5.2.0. 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.
I think the whole IsCTLFontEnabled() stuff should just go. All it does is that it hides UI elements essential for users who need them and the option to show them is buried in deep down in the options dialog without a single hint on what it actually does and we have numerous reports of users thinking LibreOffice does not support such features because of this. I don’t buy the argument that is confuses users who don’t need it, because any given user does not use like 3/4 of the application capabilities and they would be confused to death if such argument was true.
(In reply to Khaled Hosny from comment #10) > ... I don’t buy the > argument that is confuses users who don’t need it, because any given user > does not use like 3/4 of the application capabilities and they would be > confused to death if such argument was true. +1 ;)
@Khaled, (In reply to Khaled Hosny from comment #10) > I think the whole IsCTLFontEnabled() stuff should just go. All it does is > that it hides UI elements essential for users who need them and the option > to show them is buried in deep down in the options dialog without a single > hint on what it actually does and we have numerous reports of users thinking > LibreOffice does not support such features because of this. > .... So my suggestion would be to create a separate issue for this wish, assigned to libreoffice-ux-advise@lists.freedesktop.org