D19621: ViewPrivate: Make deselection by arrow keys more handy
loh tar
noreply at phabricator.kde.org
Sun Mar 10 07:42:40 GMT 2019
loh.tar updated this revision to Diff 53574.
loh.tar set the repository for this revision to R39 KTextEditor.
loh.tar added a comment.
>> me wrote:
>>
>> 2. Please test with right to left text layout, and mixed text layout.
>
> Can't say what is to be expect in such case, so test is difficult for me
OK. It pointed out that these RTL implementation is slightly buggy in Kate/Qt and therefore I was lost. After recent "forced" request to implement it anyway and to ignore my inner aversions to do it, I found good old BUG 165397. These test data and comments helped me to judge now that this patch works as best as it is possible.
- Remove pointless "m_viewInternal->m_view->"
- Re-arrange if cases, looks now similar as rest of the code
- Use QFETCH, but not for all needed data. Requests to fix that will ignored :p
REPOSITORY
R39 KTextEditor
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D19621?vs=53540&id=53574
REVISION DETAIL
https://phabricator.kde.org/D19621
AFFECTED FILES
autotests/src/kateview_test.cpp
autotests/src/kateview_test.h
src/view/kateview.cpp
To: loh.tar, #ktexteditor
Cc: cullmann, brauch, dhaumann, ngraham, kwrite-devel, kde-frameworks-devel, #ktexteditor, gennad, domson, michaelh, bruns, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190310/ad2ab26d/attachment.html>
More information about the Kde-frameworks-devel
mailing list