D13517: Fix QuickOpen with Qt 5.11 (bug 395039)

Sven Brauch noreply at phabricator.kde.org
Wed Jun 13 14:33:50 UTC 2018


brauch created this revision.
brauch added reviewers: dhaumann, cullmann.
Restricted Application added a project: Kate.
Restricted Application added a subscriber: kwrite-devel.
brauch requested review of this revision.

REVISION SUMMARY
  Apparently the item model changes in Qt 5.11 cause our quickopen list to be empty.
  
  I do not know why the old code does not work, to be honest, to me it seems okay. The only thing which is definitely wrong is the call to mapToSource; the index passed to this belongs to the model which has since been replaced, and not to the actual model attached to the proxy model. That issue is unrelated though, commenting out that code (and thus removing the feature) doesn't fix it.
  
  I like the suggested implementation better anyways, so I think we should change this in either case, but does anyone have an idea why the old code is broken? Is that maybe even a regression in Qt itself? @mwolff?

REPOSITORY
  R40 Kate

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

AFFECTED FILES
  kate/katequickopen.cpp

To: brauch, dhaumann, cullmann
Cc: kwrite-devel, mwolff, michaelh, kevinapavew, ngraham, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180613/590e1d78/attachment.html>


More information about the KWrite-Devel mailing list