<table><tr><td style="">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/D6839" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D6839#127807" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D6839#127807</a>, <a href="https://phabricator.kde.org/p/brauch/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@brauch</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Hmm, I'm still confused why this code was there then. Looking at git history, it was also changed several times, fixing issues.</p></div>
</blockquote>

<p>Yes, quite mysterious :) Perhaps the action id "vcsannotate_current_document" had been added to some personal XMLGUI rc files only? Because from how I understand this code, the action is only added to the actioncollection of the document controller, so to be appear in the UI it needs to be referrenced from somewhere, either manually by code getting the action by id and putting it somewhere, or by having the id listed in some rc file. But I could not find anything in history (or even in the default ui_standards.rc.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Does invoking the action via its shortcut still work?</p></blockquote>

<p>Question is: which shortcut? There is none defined in the code, both in the initial and the other versions I saw in history?</p>

<p>Having a shortcut would be nice though, agreed. Both for showing and hiding the annotation border even. Which shortcut keys were you thinking off?</p>

<p>Though ideally this action & shortcut would be provided and handled by the vcs plugins themselves . So to restore any shortcut functionality you remember, I would be up to write a patch to allow<br />
vcs plugins to hook up such action with shortcut to the main menu, with bonus points also for making it a toggle action depending on the status, independent of being involed by context menu (provided completely by plugins) or main menu.</p>

<p>But in any case, the code as removed in this patch currently is de-funct code, unless I really missed something.<br />
Also slightly hacky how it uses a full VcsPluginHelper instance. So instead of investing into restoring any functionality this code here should have provided, I am rather desperate to rewrite some things in general, so document controller does not need to know about vcs, and vcs plugins properly support multiple views on the same document with/without annotation border. And then with that experience  hopefully also try to work on finally getting the annotation border to update on changes. When fixing things for <a href="https://phabricator.kde.org/D6838" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D6838</a> I already collected some ideas...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6839" rel="noreferrer">https://phabricator.kde.org/D6839</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop, brauch, apol<br /><strong>Cc: </strong>kdevelop-devel<br /></div>