Bug 390585

Summary: [Kicker/Kickoff] "Pin to Task Manager" causes the menu to close
Product: [Plasma] plasmashell Reporter: Nate Graham <nate>
Component: Application Launcher (Kickoff)Assignee: Alexander Lohnau <alexander.lohnau>
Status: RESOLVED FIXED    
Severity: normal CC: alexander.lohnau, bugseforuns, epost.kde, plasma-bugs
Priority: NOR    
Version: master   
Target Milestone: 1.0   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In: 5.20
Sentry Crash Report:
Attachments: Video of the problem happening with Kicker
Video of the problem happening with Kickoff

Description Nate Graham 2018-02-16 21:00:20 UTC
When you right-click on an app listed in Kicker or Kickoff and choose "Pin to Task Manager", Kicker or Kickoff will close. This is annoying if you're trying to pin multiple apps in a row, and seems unnecessary, since closing the menu manually is a trivially simple matter.
Comment 1 David Edmundson 2018-02-16 22:09:58 UTC
Works for me.

Are you sure?
Comment 2 Nate Graham 2018-02-16 22:14:00 UTC
Created attachment 110736 [details]
Video of the problem happening with Kicker

Pretty sure, yeah. Unless my eyes are lying to me!
Comment 3 Nate Graham 2018-02-16 22:14:22 UTC
Created attachment 110737 [details]
Video of the problem happening with Kickoff
Comment 4 Nate Graham 2018-02-16 22:15:32 UTC
(Plasma 5.12.1, of course)
Comment 5 Øystein Steffensen-Alværvik 2018-06-24 22:27:21 UTC
Reproducable on my end too.
Comment 6 Bug Janitor Service 2020-08-13 07:37:43 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/210
Comment 7 David Edmundson 2020-08-13 08:49:23 UTC
davidedmundson wrote:
> @bug_id = 390585
> @bug_status = RESOLVED
> @resolution = FIXED
> @cf_commitlink = https://invent.kde.org/plasma/plasma-workspace/commit/1d4cb4857d5a401d293ea54cb35b3a4534da5f1c
> @cf_versionfixedin = 5.20
>
> Git commit 1d4cb4857d5a401d293ea54cb35b3a4534da5f1c by David Edmundson, on behalf of Alexander Lohnau.
> Committed on 13/08/2020 at 08:26.
> Pushed by alex into branch 'master'.
>
> Do not close Kicker when pinning entry
> FIXED-IN: 5.20
>
> When we add an app to the panel/desktop/taskmanager we don't want
> to close Kicker.
>
> M  +1    -1    applets/kicker/plugin/appentry.cpp
> M  +1    -1    applets/kicker/plugin/runnermatchesmodel.cpp