Bug 432879 - Emoji picker should remember the window position
Summary: Emoji picker should remember the window position
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Emoji Selector (show other bugs)
Version: 5.21.0
Platform: Other Linux
: NOR wishlist
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2021-02-13 07:12 UTC by Dan Dascalescu
Modified: 2024-09-20 03:26 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 6.3.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Dascalescu 2021-02-13 07:12:52 UTC
Every time I launch the emoji picker, it shows up in the upper right corner of the screen. I have to force its KDE window settings to make it move to the center, but that's not a reliable solution for most situations. Ideally, it should remember its previous positinn and display itself at those coordinates.
Comment 1 Nate Graham 2021-02-14 14:54:35 UTC
Needs to manually call the relevant kconfig functions because it doesn't use KXMLGui which provides this functionality automatically.

Perhaps we need some kind of QML-based convenience component that will implement this functionaluty when added.
Comment 2 Dan Dascalescu 2021-02-21 08:59:24 UTC
Updating to confirm the issue is still there in Plasma 5.21, which made most windows preserve their size and position by default (vs. 5.18.5).
Comment 3 Nate Graham 2021-02-22 00:11:04 UTC
The change to do that actually happened in a Frameworks update (5.74 IIRC), and most apps got the updated behavior automatically because they use a KXMLGui-provided widget as their main window, and that widget got the new feature automatically.

But the Emoji picker window is not a QWidgets app, so it does not use KXMLGui, so it did not get the feature automatically, and it needs to be manually updated to implement it.
Comment 4 Nate Graham 2024-09-09 22:20:32 UTC
Ooooh I can do this now. Will do soon.
Comment 5 Bug Janitor Service 2024-09-10 05:32:51 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2511
Comment 6 Nate Graham 2024-09-20 03:26:01 UTC
Git commit 915856add537dcbaa0203f30060809a2662f4f71 by Nate Graham.
Committed on 20/09/2024 at 03:16.
Pushed by ngraham into branch 'master'.

emojier: adopt WindowStateSaver

This way the window will remember its size and (on X11) position.
FIXED-IN: 6.3.0

M  +7    -0    emojier/app/ui/Emojier.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/915856add537dcbaa0203f30060809a2662f4f71