D8822: Rename inline: scrolling causes rename of wrong file

Andreas Krutzler noreply at phabricator.kde.org
Wed Nov 15 17:34:05 GMT 2017


akrutzler added inline comments.

INLINE COMMENTS

> ngraham wrote in kitemlistroleeditor.cpp:61
> Why is this new function necessary?

Actually it's not necessary. Just my personal preference since `KIO::encodeFileName(toPlainText()` is used 3 times (2 in kitemlistroleeditor.cpp and 1 in kstandarditemlistwidget.cpp). But I can also remove this function.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D8822

To: akrutzler, ngraham, rkflx
Cc: #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171115/08d12fce/attachment.htm>


More information about the kfm-devel mailing list