| Summary: | Unhide text direction cell/paragraph attribute | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Yousuf Philips (jay) (retired) <philipz85> |
| Component: | LibreOffice | Assignee: | Yousuf Philips (jay) (retired) <philipz85> |
| Status: | RESOLVED FIXED | ||
| Severity: | enhancement | CC: | bubli, cno, fitojb, h3734236, heiko.tietze, khaled, momonasmon, samuel.mehrbrodt, vsfoote |
| Priority: | medium | ||
| Version: | 5.2.0.0.alpha0+ | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:5.2.0 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 43808, 107642, 108681 | ||
| Attachments: | format cells dialog | ||
|
Description
Yousuf Philips (jay) (retired)
2016-01-19 13:11:22 UTC
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 |