Bug 395615 - Context menu from an entry in the task manager shows "pin" instead "unpin" when the app is already pinned
Summary: Context menu from an entry in the task manager shows "pin" instead "unpin" wh...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Task Manager and Icons-Only Task Manager (show other bugs)
Version: 5.16.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-19 15:35 UTC by Patrick Silva
Modified: 2019-06-17 13:02 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.17.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2018-06-19 15:35:31 UTC
open some app that is not pinned to the task manager yet
right click its entry in the task manager, select "pin"
right click the same entry again: context menu shows "pin" (checked) instead "unpin"
Comment 1 Kai Uwe Broulik 2018-06-27 14:45:54 UTC
It has a checkbox next to it, so this is correct. It's somewhat inconsistent with launchers, though, that use a separate "Unpin" action.
Comment 2 Eike Hein 2018-06-27 20:05:51 UTC
Yeah, we should consistify this.
Comment 3 Nate Graham 2019-06-16 17:26:50 UTC
Here's a patch: https://phabricator.kde.org/D21856
Comment 4 Nate Graham 2019-06-17 13:02:45 UTC
Git commit af9a794695c73e89a7f0958bd7f071d3fec02996 by Nate Graham.
Committed on 17/06/2019 at 13:02.
Pushed by ngraham into branch 'master'.

[Task manager] Use consistent menu items for "pin/unpin" actions

Summary:
Make the pinning-relatedmenu items consistent by showing separate Pin and Unpin menu
items when not using activities. When using activities, it's unavoidable, so we still
show a more complicated UI there with checkboxes for the per-activity pinning options.
FIXED-IN: 5.17.0

Test Plan:
Right-click on pinned launchers, open unpinned tasks, and open pinned tasks - see that all have the correct menu items

Do it again with mode than one Activity and see that all is still well

Reviewers: #plasma, #vdg, hein

Reviewed By: #plasma, hein

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D21856

M  +5    -4    applets/taskmanager/package/contents/ui/ContextMenu.qml

https://commits.kde.org/plasma-desktop/af9a794695c73e89a7f0958bd7f071d3fec02996