D22287: [colorcorrection] Introduce toggle Night Color shortcut

David Edmundson <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> zzag wrote in manager.cpp:176
> That's a copy-paste thing.

It's not. There's a difference.

one calls

  updateGlobalShortcut(KGlobalAccelPrivate::DefaultShortcut | KGlobalAccelPrivate::ActiveShortcut);

the other

  updateGlobalShortcut(KGlobalAccelPrivate::ActiveShortcut);

but it doesn't really matter in this case

REPOSITORY
  R108 KWin

BRANCH
  arcpatch-D22287

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

To: zzag, #kwin, davidedmundson
Cc: romangg, ngraham, broulik, davidedmundson, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.