SUMMARY The "Show in activities" Menu looks different between kwin and plasma (e.g. the icons-only taskbar). Kwin uses a radio button for "all activites" and checkboxes for single activities and shows activity icons, the taskbar does not show icons and uses checkboxes for "all activities" and the single activity menu items. I think the kwin menu should be used for the taskbar menu as well.
True that! Not a big deal but might be worth addressing along with the need for the feature parity on the "Show in activities" for kwin arising from: https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/231
Another thought: Every menu that contains checkboxes should stay open when you click a checkbox, because you may want to (un)check more than one.
This was fixed recently; forward-duping to Bug 445664. *** This bug has been marked as a duplicate of bug 445664 ***
(In reply to Nate Graham from comment #3) > This was fixed recently; forward-duping to Bug 445664. > > *** This bug has been marked as a duplicate of bug 445664 *** Are you sure, the consistency part is addressed? I am not sure in which version I can test (is it in Debian testing already?), but the screenshots in the pull request don't look like it's consistent between the panel and kwin.
No, the consistency issues were not addressed. The feature requested was added but not in exactly the same way. They can't share code unfortunately. However the feature does exist in both places now.
I am unsure what's the current state now. But what I would like to see for usability and consistency are these points: - KWin and the different task bar implementations should use the same or a similar menu - "All activities" should be a radiobutton or maybe just a button/menu entry, as it is not an option, but controls other options, i.e., checking "all" checks all others and unchecking one other unchecks "all". - When adding/removing the window from activities, the menu should stay open. When I have five activities and want to assign the window to three other activities, I currently need to open the menu three times to assign it to other activities and a fourth time to remove it from the current activity.
Let's please try to keep a bug reports about a single thing. When bug reports ask for or discuss multiple things, then they become messy and un-actionable.
This bug report always just was about one thing: Consistency of the menus. That's why I don't understand (as long as the patch didn't involve the suggested UX change, what it does not according to the screenshots in the pull request) yet why it was closed together with a bugreport about a not functioning menu, what never was the issue here (and for me both menus always worked and I only suggesting improving the UX a bit).
OK, re-opening. Ideally they would share code so that consistency could be achieved truly automatically. That's hard though because one lives in KWin and is a QMenu, and the other lives in Plasma and is a QML-based menu.
Fixed in https://invent.kde.org/plasma/kwin/-/commit/c0447c36813d404d82b801073f961b250d669d36
In Plasma 5.27.2 the menus work as expected, but the title bar menu has icons and the panel menu does not have icons. I reopen here, as the bug was about consistency and talking about identical menus, which still has this minor issue that can probably be fixed easily.
Got it, will fix.
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/771
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2825
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/1473
Those three ought to do it. Unfortunately they'll have to be merged only for Plasma 6; we can't backport to Plasma 5.27.5
Fine! No needs for backports. The menu works as it should and was only slightly inconsistent.
Git commit b12a60c15ee99a7c06b6d9a4fd0680d74cb4f35a by Nate Graham. Committed on 13/04/2023 at 16:46. Pushed by ngraham into branch 'master'. applets/taskmanager: show Activity icons in relevant context menu items The window decoration menu does; the equivalent Task Manager menus ought to do the same. Depends on https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2825 FIXED-IN: 6.0 M +7 -4 applets/taskmanager/package/contents/ui/ContextMenu.qml https://invent.kde.org/plasma/plasma-desktop/commit/b12a60c15ee99a7c06b6d9a4fd0680d74cb4f35a
Git commit 0c374b486a5e717be01d1e4cbe3ac6ed94a64b6e by Nate Graham. Committed on 13/04/2023 at 16:46. Pushed by ngraham into branch 'master'. libtaskmanager: add activityIcon getter Needed for accessing activity icons from the Task Manager. M +10 -0 libtaskmanager/activityinfo.cpp M +8 -0 libtaskmanager/activityinfo.h https://invent.kde.org/plasma/plasma-workspace/commit/0c374b486a5e717be01d1e4cbe3ac6ed94a64b6e
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2826
Git commit 56e25cce0954970600c4128a236ba4895f8c5e7a by Nate Graham. Committed on 13/04/2023 at 18:00. Pushed by ngraham into branch 'Plasma/5.27'. applets/taskmanager: show Activity icons in relevant context menu items The window decoration menu does; the equivalent Task Manager menus ought to do the same. Depends on https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2825 FIXED-IN: 6.0 (cherry picked from commit b12a60c15ee99a7c06b6d9a4fd0680d74cb4f35a) M +7 -4 applets/taskmanager/package/contents/ui/ContextMenu.qml https://invent.kde.org/plasma/plasma-desktop/commit/56e25cce0954970600c4128a236ba4895f8c5e7a
Git commit e3f688661309e732baa540c9bbb19476307f267e by Nate Graham. Committed on 13/04/2023 at 17:54. Pushed by ngraham into branch 'Plasma/5.27'. libtaskmanager: add activityIcon getter Needed for accessing activity icons from the Task Manager. (cherry picked from commit 0c374b486a5e717be01d1e4cbe3ac6ed94a64b6e) M +10 -0 libtaskmanager/activityinfo.cpp M +8 -0 libtaskmanager/activityinfo.h https://invent.kde.org/plasma/plasma-workspace/commit/e3f688661309e732baa540c9bbb19476307f267e
In fact I was wrong; it was backportable and the fix will show up in Plasma 5.27.5