| Summary: | Panel and panel widgets are too easy to accidentally delete | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | homem.gustavo |
| Component: | Panel | Assignee: | Nate Graham <nate> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | nate, rikmills |
| Priority: | NOR | Keywords: | usability |
| Version: | 5.18.4 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://commits.kde.org/plasma-desktop/ba66d2eabe38bc357d799bff391cf5edbce139c0 | Version Fixed In: | 5.18.5 |
| Sentry Crash Report: | |||
| Attachments: |
The plasma panel is too easy to misconfigure by accident.
The plasma panel is too easy to misconfigure by accident II The plasma panel is too easy to misconfigure by accident III |
||
Created attachment 127393 [details]
The plasma panel is too easy to misconfigure by accident II
Created attachment 127394 [details]
The plasma panel is too easy to misconfigure by accident III
Those are very reasonable concerns and suggestions IMO. Working on this. Patch submitted: https://phabricator.kde.org/D28710 @Nate, Great work on the submitted patch. I just want to note that inside the "More options" menu the "Remove panel" option is still the nearest one to the mouse cursor. Maybe it could be not the first? (just a suggestion). However it turns out, it is fore sure a great improvement over the current situation. Thanks again. Oh duh, good point. Changed in the patch. Thanks again Nate. Any idea if this this be on time for Kubuntu 20.04? Currently that'll be up to whether the Kubuntu people decide to cherry-pick the change, as Kubuntu will ship with Plasma 5.18 and the patch is targeted for Plasma 5.19 because it changes the UI. It's possible that this could be considered a bugfix as it doesn't add any new strings, in which case it can land for a 5.18 bugfix release, which would get it into Kubuntu with no extra packaging necessary. I can see about that. I think Kubuntu 20.04 will be a reference release for KDE, and therefore preventing accidental misconfigurations is important. Furthermore the UI changes that removed Lock/Unlock widgets are not minor and were done so near the release date that I think the changes now introduced by your patch are quite justifiable. Git commit ba66d2eabe38bc357d799bff391cf5edbce139c0 by Nate Graham.
Committed on 13/04/2020 at 17:05.
Pushed by ngraham into branch 'Plasma/5.18'.
Make accidental panel and panel widget deletion more difficult
Summary:
We make it very easy to delete panels and panel widgets--perhaps a bit too easy. User
feedback has indicated that we may need to re-work this a bit, and that perhaps D20144 was a
mistake.
This patch moves the {nav Remove Panel} button back into the {nav More} menu where it used to
be, to make it harder to accidentally click on. It also moves the {nav Remove} button for
panel widgets higher up in the menu so it's not so close to the cursor.
To adjust to these changes, the layout of buttons on the panel toolbar is changed to
preserve visual balance, and the {nav More} menu's name is adjusted to reflect the fact that
removing a panel is not a setting.
FIXED-IN: 5.18.5
Test Plan: {F8227561}
Reviewers: #vdg, #plasma, ndavis
Reviewed By: #vdg, ndavis
Subscribers: rikmills, broulik, plasma-devel
Tags: #plasma
Maniphest Tasks: T12942
Differential Revision: https://phabricator.kde.org/D28710
M +20 -2 containments/panel/contents/ui/ConfigOverlay.qml
M +19 -0 desktoppackage/contents/configuration/panelconfiguration/MoreSettingsMenu.qml
M +15 -26 desktoppackage/contents/configuration/panelconfiguration/ToolBar.qml
https://commits.kde.org/plasma-desktop/ba66d2eabe38bc357d799bff391cf5edbce139c0
(In reply to Nate Graham from comment #10) > Currently that'll be up to whether the Kubuntu people decide to cherry-pick > the change, as Kubuntu will ship with Plasma 5.18 and the patch is targeted > for Plasma 5.19 because it changes the UI. > > It's possible that this could be considered a bugfix as it doesn't add any > new strings, in which case it can land for a 5.18 bugfix release, which > would get it into Kubuntu with no extra packaging necessary. I can see about > that. I certainly can't cherry pick what is arguably a UI feature change this late for 20.04. It is also somewhat doubtful that this would be acceptable as a 'bugfix' for the 5.18.5 SRU update in a few weeks. |
Created attachment 127392 [details] The plasma panel is too easy to misconfigure by accident. SUMMARY The plasma panel is too easy to misconfigure by accident. STEPS TO REPRODUCE 1. Right click on the panel 2. Verify that removing the panel altogether is one click away without confirmation 3. Verify that remove parts of the panel (ex: system tray, task bar) is available as the nearest option to the mouse cursor upon right click OBSERVED RESULT It is rather easy for users to accidentally break the initial configuration setup by system integrators. EXPECTED RESULT Would be great to have the same options available but in a way that protects from accidental changes. For example: * the option to remove the panel could be hidden in the "more settings" menu (like it was in the past) or could require a user confirmation * the option to remove panel components could perhaps be on top of the context menu rather than on the bottom SOFTWARE/OS VERSIONS Linux/KDE Plasma: (available in About System) KDE Plasma Version: 5.18.4 KDE Frameworks Version: 5.68.0 Qt Version: 5.12.5 ADDITIONAL INFORMATION