D27404: Hide empty top-level menu items
Aleksei Nikiforov
noreply at phabricator.kde.org
Fri Feb 14 15:17:10 GMT 2020
alnikiforov created this revision.
alnikiforov added reviewers: yurchor, mlaurent, ltoscano.
Herald added a project: Documentation.
Herald added a subscriber: kde-doc-english.
alnikiforov requested review of this revision.
REVISION SUMMARY
If there's no data, it looks like it's bugged. Let's hide it in such case.
Similar to: D27403 <https://phabricator.kde.org/D27403>
TEST PLAN
1. Start khelpcenter
2. Open every top level menu item
3. Every top level item shouldn't be empty
For me, top level items "Konqueror Settings Modules" and "Other Settings Modules" are empty. With this change they'd be hidden instead.
REPOSITORY
R125 KHelpCenter
REVISION DETAIL
https://phabricator.kde.org/D27404
AFFECTED FILES
khelpcenter/navigator.cpp
To: alnikiforov, yurchor, mlaurent, ltoscano
Cc: kde-doc-english, gennad, fbampaloukas, skadinna
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20200214/339a3c1c/attachment.html>
More information about the kde-doc-english
mailing list