loh.tar created this revision.
loh.tar added a reviewer: KTextEditor.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
loh.tar requested review of this revision.
REVISION SUMMARY
The only two included changes noticeable by the user are:
- Use line number color for unfolded triangle to be less intrusive
- Move the line modification hint away from the edit area between the line
numbers and the folding area to be less irritating when cursor is in first
column
Most of the changes in this patch could you see as code cosmetic or an
attempt to make it more maintenance-friendly.
- Cleanup, avoid redundancies and nesting
- Rename members, make them unified named
- Give each area an own value name
- Remove m_oldBackgroundColor, can't see any benefit
- Make positionToArea() and sizeHint() less error prone in case of design
changes
TEST PLAN
Pls try it, but I'm not sure if I'm done with it
F6783737: 1555761488.png <https://phabricator.kde.org/F6783737>
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D20697
AFFECTED FILES
src/view/kateviewhelpers.cpp
src/view/kateviewhelpers.h
To: loh.tar, #ktexteditor
Cc: kwrite-devel, kde-frameworks-devel, #ktexteditor, domson, michaelh,
ngraham, bruns, demsking, cullmann, sars, dhaumann
