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

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/plasma-workspace/-/commi
                   |                            |t/bbe6ff3cffa567f7c4bc568fb
                   |                            |a5d89dc445a153a

--- Comment #54 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit bbe6ff3cffa567f7c4bc568fba5d89dc445a153a by David Edmundson.
Committed on 06/02/2025 at 14:18.
Pushed by davidedmundson into branch 'master'.

klipper: Avoid breaking Listview

A listview by default will keep the highlight in view, scrolling the
list appropriately. This is not desired in Klipper where putting the
mouse over a clipped final delegate should not move the index.

A complicated fix was added in 31a547cc by trying to tweak at what point
we change the current index.

This is extremely messy especially when coupled with delegates of
changing heights. We start changing the current item which we know will
trigger a scroll which changes the current item. ListView gets confused
and it's not surprising.

There's a considerably simpler fix for the original bug, don't use
ListView highlights if we don't want ListView behaviour.

This fixes overlapping delegates in Klipper.

M  +12   -10   klipper/declarative/qml/ClipboardItemDelegate.qml
M  +1    -3    klipper/declarative/qml/ClipboardMenu.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/bbe6ff3cffa567f7c4bc568fba5d89dc445a153a

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

Reply via email to