<table><tr><td style="">davidedmundson accepted this revision.<br />davidedmundson added a comment.<br />This revision is now accepted and ready to land.
</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 #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></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>