Remove warning before hidding the MenuBar
AbandonedPublic

Authored by tcanabrava on Jan 28 2019, 10:49 AM.

Details

Reviewers
ngraham
Summary

No other KDE app does this, while also allowing to show / hide
the menubar. This actualy seemed quite extreme as there's no way
to say "ok, don't annoy me again". And since konsole, dolphin,
digikam, etc just hide / show the menubar, do the same.

Diff Detail

Repository
R106 KSysguard
Branch
removeUneededStrings
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7614
Build 7632: arc lint + arc unit
tcanabrava created this revision.Jan 28 2019, 10:49 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 28 2019, 10:49 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
tcanabrava requested review of this revision.Jan 28 2019, 10:49 AM
ngraham requested changes to this revision.Jan 28 2019, 3:50 PM
ngraham added a subscriber: ngraham.

No other KDE app does this

That's not true. Kate and Gwenview do this too, and for good reason: without this warning, there's no way to know how to get the menu bar back again if you remove it by accident and/or don't know about the ctrl+m shortcut.

Konsole doesn't show a warning because it's a power user app; presumably the user knows what he/she is doing. Dolphin doesn't have this warning because it automatically creates a menu-like toolbar button when the menubar is hidden, which is an acceptable alternative. But for apps that don't do this or something else (e.g. putting the menubar in the titlebar instead) we need to show an ugly warning like this.

This revision now requires changes to proceed.Jan 28 2019, 3:50 PM

No other KDE app does this

That's not true. Kate and Gwenview do this too, and for good reason: without this warning, there's no way to know how to get the menu bar back again if you remove it by accident and/or don't know about the ctrl+m shortcut.

Konsole doesn't show a warning because it's a power user app; presumably the user knows what he/she is doing. Dolphin doesn't have this warning because it automatically creates a menu-like toolbar button when the menubar is hidden, which is an acceptable alternative. But for apps that don't do this or something else (e.g. putting the menubar in the titlebar instead) we need to show an ugly warning like this.

I can argue that the ksysguard is also a power user app, but I'll now argue something else:
The hidding / displaying the menus should not be the *app* responsability, but KXmlGuiWindow. If this code is moved there, all apps that are currently doing this manually can benefit the feature and have a tiny bit less code.

The hidding / displaying the menus should not be the *app* responsability, but KXmlGuiWindow. If this code is moved there, all apps that are currently doing this manually can benefit the feature and have a tiny bit less code.

Yep, strongly agree.

tcanabrava abandoned this revision.Mar 28 2019, 11:44 AM