Hide empty top-level menu items
Needs ReviewPublic

Authored by alnikiforov on Fri, Feb 14, 3:17 PM.

Details

Summary

If there's no data, it looks like it's bugged. Let's hide it in such case.

Similar to: 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.

Diff Detail

Repository
R125 KHelpCenter
Lint
Lint Skipped
Unit
Unit Tests Skipped
alnikiforov created this revision.Fri, Feb 14, 3:17 PM
Restricted Application added a project: Documentation. · View Herald TranscriptFri, Feb 14, 3:17 PM
Restricted Application added a subscriber: kde-doc-english. · View Herald Transcript
alnikiforov requested review of this revision.EditedFri, Feb 14, 3:17 PM

Here's how empty "Other Settings Modules" entry looks when selected: