D28744: Rewrite of the global shortcuts kcm

[Nathaniel Graham]

ngraham added a comment.


  In D28744#659600 <https://phabricator.kde.org/D28744#659600>, @davidre wrote:
  
  > In D28744#659447 <https://phabricator.kde.org/D28744#659447>, @ngraham wrote:
  >
  > > I find that I'm not able to set shortcuts involving the space key. The key sequence get repeated twice in the button and the Apply button ever becomes enabled. Can you reproduce?
  >
  >
  > That's the current behavior of KeySequenceItem and KKeySequenceWidget. You should be able to observe it everywhere such an item is used
  
  
  Ah, I can indeed reproduce that with other instances of that in QML software (e.g. applet config windows' shortcut pages). I think we need that fixed before this lands as otherwise it would be a regression compared to the existing KCM.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  kcmkeys2 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D28744

To: davidre, #vdg, #plasma, ngraham
Cc: GB_2, broulik, davidedmundson, nicolasfella, ngraham, iasensio, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200429/20eebf9b/attachment.html>