[Konsole-devel] Review Request: When using RTL interface, do not make terminal emulator RTL

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102976/#review8328
-----------------------------------------------------------


This review has been submitted with commit 044fdf90200fd9ac11546783e7e3de727c1ba13b by Jekyll Wu to branch master.

- Commit Hook


On Nov. 16, 2011, 11:53 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102976/
> -----------------------------------------------------------
> 
> (Updated Nov. 16, 2011, 11:53 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> Konsole does contain the code for forcing LTR in the terminal area :
> 
>     QWidget::setLayoutDirection(Qt::LeftToRight);
> 
> However, according to the Qt doc[1], that method does not influence text layout direction anymore since Qt-4.7. So this problem is basically a regression caused by upstream change. I am not sure about that statement because I have never used RTL before and have no idea how konsole behaved with old version of Qt.
> 
> One solution is to force QPainter to use LTR when drawing text. This is what this patch does.
> 
> Since I am definitely no expert on this RTL issue, please do not take above analysis too seriously. Is there any expert we can refer to?
> 
> 
> This addresses bug 279048.
>     http://bugs.kde.org/show_bug.cgi?id=279048
> 
> 
> Diffs
> -----
> 
>   src/TerminalDisplay.cpp 46c6438 
> 
> Diff: http://git.reviewboard.kde.org/r/102976/diff/diff
> 
> 
> Testing
> -------
> 
> Using the attached text for basic testing. The result seems OK to me when forcing LTR to the terminal area
> 
> And according to the the bug reporter who seems to use Hebrew as native language, the patch solves the problem and no regression is observed so far.
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20111120/f0eb02ee/attachment.html>