<div dir="ltr"><div><div><div><div><div>As with most bugs it's two people doing something a slightly wrong that makes something actually wrong.<br><br>The documentation explitcitly says:<br><br>>shortcut : QKeySequence<br>>This property holds the _*mnemonic*_ associated with the button.<br><br></div><div>(emphasis mine)<br></div><div><br><a href="http://doc.qt.io/qt-5/qabstractbutton.html#shortcut-prop">http://doc.qt.io/qt-5/qabstractbutton.html#shortcut-prop</a><br><br>Your code isn't setting it to the mnemonic. It's setting it to something unrelated.<br>So even though it seems to work /technically/ you're going against the documentation.<br> <br>Regardless of whether KDE code existed or not, you'd still have a bug if you ever call setText later with a mnemonic or switched languages.<br><br>> So consider me sceptical to the way it now affects everything :)</div><br></div>So consider me equally sceptical that we have any duty to fix it :)<br></div><br><br>I tried to change qabstractbutton to have two shortcuts, but the problem there is I potentially change the behaviour of the QAbstractButton::shortcut accessor. Any thoughts on how to avoid that?<br><br></div>If we did want to make our code affect only KDE apps we could potentially check the qApp->organisationDomain? KDE people  - would that work?<br><br></div>David<br></div>