This revision was automatically updated to reflect the committed changes.
Closed by commit R104:eacbddd22d99: Show config selection OSD when display
button is pressed (authored by sebas).
REPOSITORY
R104 KScreen
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10896?vs=28251&id=28312
dvratil accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R104 KScreen
BRANCH
sebas/displaybutton
REVISION DETAIL
https://phabricator.kde.org/D10896
To: sebas, #plasma, dvratil
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts,
s
sebas updated this revision to Diff 28251.
sebas added a comment.
- clean up now-dead code
REPOSITORY
R104 KScreen
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D10896?vs=28202&id=28251
BRANCH
sebas/displaybutton
REVISION DETAIL
https://phabricator.kde.org/D10896
AFFECTED F
sebas added a comment.
OsdManager::showActionSelector() will delete existing OSDs before showing a
new one, so we won't create multiple OSDs. That's arguably not as nice as
tracking if it's already shown, but I think I'd rather avoid the added
complexity of adding a check if it's already the
dvratil requested changes to this revision.
dvratil added a comment.
This revision now requires changes to proceed.
Hmm, I guess you can now kill `m_buttonTimer`, `m_iteration`,
`resetDisplaySwitch()` and `applyGenericConfig()` as well, as those are used
exclusively by the config iteration.
ngraham retitled this revision from "show config selection osd when display
button is pressed" to "Show config selection OSD when display button is
pressed".
ngraham edited the summary of this revision.
REPOSITORY
R104 KScreen
REVISION DETAIL
https://phabricator.kde.org/D10896
To: sebas, #
sebas created this revision.
sebas added reviewers: Plasma, dvratil.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
sebas requested review of this revision.
REVISION SUMMARY
This makes the working of the display button much more intuitive