Bug 439269 - Opening the settings dialog closes the applet even if it's pinned open
Summary: Opening the settings dialog closes the applet even if it's pinned open
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Digital Clock (show other bugs)
Version: 5.22.1
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-28 17:13 UTC by popov895
Modified: 2021-07-22 16:24 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.22.3


Attachments
Bug (55.78 KB, image/png)
2021-06-29 10:19 UTC, popov895
Details

Note You need to log in before you can comment on or make changes to this bug.
Description popov895 2021-06-28 17:13:59 UTC
SUMMARY

Opening the settings dialog closes the applet even if it's pinned open.

STEPS TO REPRODUCE

1. Open Digital Clock applet
2. Check "Keep Open" button in the top right corner
3. Open Settings dialog (clicking "Configure Digital Clock..." button or menuitem with the same name in the context menu)

SOFTWARE/OS VERSIONS

Operating System: openSUSE Tumbleweed 20210626
KDE Plasma Version: 5.22.1
KDE Frameworks Version: 5.83.0
Qt Version: 5.15.2
Graphics Platform: X11
Comment 1 Felipe Kinoshita 2021-06-29 07:05:22 UTC
Can confirm.
Comment 2 Bug Janitor Service 2021-06-29 07:48:31 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/517
Comment 3 popov895 2021-06-29 10:19:37 UTC
Created attachment 139732 [details]
Bug

Do you also have this bug after applying your patch?
Comment 4 popov895 2021-06-29 12:04:00 UTC
> Do you also have this bug after applying your patch?

Looks like it's fixed in Plasma 5.22.2.
Comment 5 David Redondo 2021-07-06 09:46:55 UTC
Git commit 963790d55da28a2e3a61e24ba62e67becccf1ec6 by David Redondo.
Committed on 06/07/2021 at 09:46.
Pushed by davidre into branch 'master'.

Don't hide expanded representation if plasmoid is pinned
FIXED-IN:5.22.3

M  +5    -1    desktoppackage/contents/applet/CompactApplet.qml

https://invent.kde.org/plasma/plasma-desktop/commit/963790d55da28a2e3a61e24ba62e67becccf1ec6
Comment 6 David Redondo 2021-07-06 09:48:03 UTC
Git commit 0ac0bbc8f297a7f5abe5bc71825b4c7cfcf65271 by David Redondo.
Committed on 06/07/2021 at 09:47.
Pushed by davidre into branch 'Plasma/5.22'.

Don't hide expanded representation if plasmoid is pinned
FIXED-IN:5.22.3


(cherry picked from commit 963790d55da28a2e3a61e24ba62e67becccf1ec6)

M  +5    -1    desktoppackage/contents/applet/CompactApplet.qml

https://invent.kde.org/plasma/plasma-desktop/commit/0ac0bbc8f297a7f5abe5bc71825b4c7cfcf65271
Comment 7 popov895 2021-07-15 07:33:35 UTC
> Looks like it's fixed in Plasma 5.22.2.
I was wrong. It's reproducible on KDE neon Unstable Edition (Plasma 5.22.80, Frameworks 5.85.0)
Comment 8 Felipe Kinoshita 2021-07-15 14:10:47 UTC
It's working for me now