...to avoid a deceptive look
With recent fixes was also the closing brace folded which can cause a false look of the remaining visible code
Details
Details
- Reviewers
cullmann dhaumann ngraham - Group Reviewers
KTextEditor
Fold differend kind of code, only tested yet with
- C/C++ where it seems to work.
- Python seems to work too
Diff Detail
Diff Detail
- Lint
Lint Skipped - Unit
Unit Tests Skipped
Comment Actions
I like the idea, its nice for e.g. if then else cascades, you get then out of stuff like
if (blockwise || range.onSingleLine()) {
lineStart = range.start().column();
remainingChars = range.columnWidth();
} else if (i == range.start().line()) {
lineStart = range.start().column();
} else if (i == range.end().line()) {
remainingChars = range.end().column();
}
if (blockwise || range.onSingleLine()) {
} else if (i == range.start().line()) {
} else if (i == range.end().line()) {
}which is still nicely readable.
Are others ok with this, too?
For the code: Perhaps one should only adjust at all, if the ranges spans more than one line, to avoid accidents.