D7398: Don't create QWhatsThis when user presses showContextHelp button
Martin Flöser
noreply at phabricator.kde.org
Fri Aug 18 19:02:33 UTC 2017
graesslin created this revision.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
I don't know what the idea of entering QWhatsThis mode was, but the code
is pretty much dead. There is no QWhatsThis window shown as KWin does
not have any UI or help. The window which eventually gets shown in
context help is provided by the client application.
Furthermore KWin never leaves the QWhatsThis mode. At least one code
path entering in leave is dead in general and one at least on Wayland
(but probably also on X11 as the window never gets shown).
The show context help functionality works fine without the QWhatsThis
handling, so let's remove it.
I tried to blame what was the idea behind it, but it was unchanged since
decades and blame ended in a code moving commit.
TEST PLAN
Used show context help (kcmshell5 --platform xcb kwinoptions)
REPOSITORY
R108 KWin
BRANCH
drop-whats-this
REVISION DETAIL
https://phabricator.kde.org/D7398
AFFECTED FILES
client.cpp
events.cpp
To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170818/a1060e51/attachment-0001.html>
More information about the Plasma-devel
mailing list