https://bugs.kde.org/show_bug.cgi?id=491635

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-desktop/-/commit/
                   |                            |80bbe282b227fb4c4ab8ccaed3d
                   |                            |06a1b6d64904e
             Status|ASSIGNED                    |RESOLVED

--- Comment #8 from Nate Graham <n...@kde.org> ---
Git commit 80bbe282b227fb4c4ab8ccaed3d06a1b6d64904e by Nate Graham, on behalf
of David Edmundson.
Committed on 30/08/2024 at 03:51.
Pushed by ngraham into branch 'master'.

Folderview: Avoid two way sync of positions

Positions are stored in the class Positioner. Items on screen follow this
property. When it changes a timer is started and we sync to the configuration
file.

There was existing code to also handle the configuration file changing and
re-sync back to Positioner in a circular trip. Due to the timer this isn't
always in sync if multiple moves are performed quickly and data is lost.

There is no need to update our current state on this configuration changing,
it's not something a user can manipulate externally. It seems it was added by
accident. Positioner is now the one source of truth and the configuration file
is only used for the initial load or resolution changes.

M  +0    -4    containments/desktop/package/contents/ui/FolderViewLayer.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/80bbe282b227fb4c4ab8ccaed3d06a1b6d64904e

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to