D29210: [NestedListHelper] Fix indentation of selection, add tests
David Faure
noreply at phabricator.kde.org
Sun Apr 26 23:57:11 BST 2020
dfaure added inline comments.
INLINE COMMENTS
> nestedlisthelper.cpp:225
> + if (delta > 0) {
> + // No list, we're increasing indentation -> create a new one
> + listFmt.setStyle(QTextListFormat::ListDisc);
should the opposite happen? delete the list when decreasing indentation?
Or is that what the blkFmt.setObjectIndex(-1); is about?
A comment there would be useful...
To me setObjectIndex(-1) sounds like it could kill any kind of embedded object there.... but I might be wrong.
REVISION DETAIL
https://phabricator.kde.org/D29210
To: poboiko, #frameworks, dfaure, mlaurent
Cc: kde-frameworks-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200426/3899fc90/attachment.html>
More information about the Kde-frameworks-devel
mailing list