D19188: only enable the module help button if the module actually has help
Harald Sitter
noreply at phabricator.kde.org
Wed Feb 20 15:27:00 GMT 2019
sitter created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.
REVISION SUMMARY
the module help button specifically launches the help for DocPath set in
the desktop file of the KCM, previously the code however only checked if
the KCM pragmatically disabled the help button. while supposedly its good
form for a module to explicitly disable the help button when it has no help
but since that depends on humans to remember doing that it's a bit
unreliable.
so, the module help button now only gets enabled when there actually is
module help available AND the button wasn't explicitly disabled.
this prevents the help button from getting enabled but then launching
the generic khelpcenter since no actual DocPath was set in the kcm
desktop metadata
TEST PLAN
- about (has no doc and disables button) -> disabled button
- memory (has doc) -> enabled button
- energy (has no doc but doesn't explicitly disable button) -> disabled button
REPOSITORY
R102 KInfoCenter
BRANCH
better-help
REVISION DETAIL
https://phabricator.kde.org/D19188
AFFECTED FILES
infocenter.cpp
To: sitter
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190220/7cee69c9/attachment.html>
More information about the Plasma-devel
mailing list