| 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-null> |
| 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/Implemented 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. |