D9711: Fix renamed file reclaims focus

Elvis Angelaccio noreply at phabricator.kde.org
Sun Jan 7 17:26:02 GMT 2018


elvisangelaccio added inline comments.

INLINE COMMENTS

> dolphinview.cpp:1362
> +        if (!selectionManager->hasSelection()) {
> +            const int currentIndex = m_model->index(m_currentItemUrl);
> +            if (currentIndex != -1) {

I'm not 100% convinced. Why does the model return a wrong index? If I select b.tmp, here I'd expect the index for b.tmp, not the one for a.tmp

REPOSITORY
  R318 Dolphin

BRANCH
  master

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

To: akrutzler, ngraham, michaelh, #dolphin
Cc: elvisangelaccio, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180107/8d716549/attachment.htm>


More information about the kfm-devel mailing list