<table><tr><td style="">brauch created this revision.<br />brauch added reviewers: dhaumann, cullmann.<br />Restricted Application added a project: Kate.<br />Restricted Application added a subscriber: kwrite-devel.<br />brauch requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13517">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Apparently the item model changes in Qt 5.11 cause our quickopen list to be empty.</p>

<p>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.</p>

<p>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? <a href="https://phabricator.kde.org/p/mwolff/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mwolff</a>?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13517">https://phabricator.kde.org/D13517</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kate/katequickopen.cpp</div></div></div><br /><div><strong>To: </strong>brauch, dhaumann, cullmann<br /><strong>Cc: </strong>kwrite-devel, mwolff, michaelh, kevinapavew, ngraham, demsking, cullmann, sars, dhaumann<br /></div>