<table><tr><td style="">kossebau marked an inline comment as done.<br />kossebau added a comment.
</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/D6838" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Thanks for review.</p>

<p>Just found though that  the approach in this patch for handling the model leakage ignores the case of multiple views on the same document (e.g. as used with projects where lots of classes are in a single long file) and where multiple views have annotation border visible and thus need the annotation model. Simply always deleting the model on hiding the border in one view will steal it from the other views then, not good.<br />
So dropping the model leak deletion from this patch, needs more thought, and committing the rest, to fix at least the duplicated menu entries.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>BRANCH</strong><div><div>fixDuplicatedAnnotationMenuEntries</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6838" rel="noreferrer">https://phabricator.kde.org/D6838</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop, brauch<br /><strong>Cc: </strong>kdevelop-devel<br /></div>