<table><tr><td style="">fvogt 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/D24956">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/D24956#648968" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24956#648968</a>, <a href="https://phabricator.kde.org/p/davidedmundson/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidedmundson</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>[14:12] <d_ed> DavidRedondo1: my understanding is that a system might ship "konsole opens with control+t"    . The UI allows you to remove that. This would remove the entry in kglobalshortcutsrc, but because it's still  in the system defaults file as soon as you log in again it'll add it back</p></blockquote>

<p>[14:25] <DavidRedondo1> d_ed, fvogt Apparently the runtime writes the hidden thing when a component is cleanedUp <a href="https://cgit.kde.org/kglobalaccel.git/tree/src/runtime/kserviceactioncomponent.cpp#n135" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/kglobalaccel.git/tree/src/runtime/kserviceactioncomponent.cpp#n135</a><br />
 [14:27] <DavidRedondo1> Does that fail or something when the file is not writeable?<br />
 [14:31] <DavidRedondo1> I think it fails<br />
 [14:31] <DavidRedondo1> I just tested it</p>

<p>if it is indeed broken...then we may as well just merge this.</p></div>
</blockquote>

<p>It might not be broken for imported desktop files, in which case this would be a noticable regression. Not tested though.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R268 KGlobalAccel</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24956">https://phabricator.kde.org/D24956</a></div></div><br /><div><strong>To: </strong>meven, mart, Plasma, fvogt, apol, davidedmundson<br /><strong>Cc: </strong>davidedmundson, davidre, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>