Summary: | When changing display resolutions, desktop icons should try to keep the same position as much as possible | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | tomashnyk |
Component: | Desktop icons & Folder View widget | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | fernandommuniz, hein, kdedev, nate, notmart, sonichedgehog_hyperblast00 |
Priority: | NOR | ||
Version First Reported In: | 5.24.4 | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
See Also: |
https://bugs.kde.org/show_bug.cgi?id=453141 https://bugs.kde.org/show_bug.cgi?id=490891 https://bugs.kde.org/show_bug.cgi?id=468546 https://bugs.kde.org/show_bug.cgi?id=496989 |
||
Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/cc44615b34dacc2a4acfdc82de885c943318a7f6 | Version Fixed In: | 6.3.0 |
Sentry Crash Report: | |||
Attachments: | Desktop files are still getting deaorganized when connecting to a external monitor. |
Description
tomashnyk
2022-05-02 18:37:32 UTC
This heuristic is apparently (as per resolution to https://bugs.kde.org/show_bug.cgi?id=453141 ) already implemented for widgets, so implementing it for icons should not be as hard as there would be something to build on. Related to https://bugs.kde.org/show_bug.cgi?id=490891 which describes connecting and disconnecting different displays and the icons *** Bug 494688 has been marked as a duplicate of this bug. *** *** Bug 494689 has been marked as a duplicate of this bug. *** Git commit cc44615b34dacc2a4acfdc82de885c943318a7f6 by Akseli Lahtinen. Committed on 06/11/2024 at 16:30. Pushed by akselmo into branch 'master'. Refactor icon positioner saving and loading The icon positioner saving and loading is handled by QML, which makes it a bit difficult to work with the logic in positioner CPP files. This change does the following: - Explicitly save only on user interactions - Moved the save and load of config to positioner.cpp - Easier debugging of the saving and loading functions - Still uses the JSON style of save/loading like the old one - Technically should be compatible with the old config, which was also JSON. - Makes positionerApplet accessible from FolderView qml - Various checks for checking if screen is enabled or not to avoid doing unnecessary work when screen is off - A lot of fixes to avoid any racing conditions - Removed any timers - Added various unit tests Alternative to https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2603 Related: bug 482875, bug 494874, bug 472137, bug 433867, bug 494412, bug 468619, bug 449069, bug 495047 FIXED-IN: 6.3.0 M +1 -1 containments/desktop/package/contents/ui/FolderView.qml M +1 -48 containments/desktop/package/contents/ui/FolderViewLayer.qml M +165 -26 containments/desktop/plugins/folder/autotests/positionertest.cpp M +18 -3 containments/desktop/plugins/folder/autotests/positionertest.h M +23 -7 containments/desktop/plugins/folder/foldermodel.cpp M +5 -0 containments/desktop/plugins/folder/foldermodel.h M +176 -60 containments/desktop/plugins/folder/positioner.cpp M +37 -8 containments/desktop/plugins/folder/positioner.h https://invent.kde.org/plasma/plasma-desktop/-/commit/cc44615b34dacc2a4acfdc82de885c943318a7f6 Created attachment 179165 [details]
Desktop files are still getting deaorganized when connecting to a external monitor.
(In reply to Fernando M. Muniz from comment #6) > Created attachment 179165 [details] > Desktop files are still getting deaorganized when connecting to a external > monitor. That's a different problem, which looks like bug 496989 This report is for keeping icons in the same place when a display changes resolution, rather than when a display is connected. Please follow the other report for updates. I set the version back to what it had been. Please don't change this. See the field descriptions here for an explanation https://bugs.kde.org/page.cgi?id=fields.html Thanks. |