Skip to content

klipper: also show highlight for currentItem

Christoph Wolk requested to merge cwo/plasma-workspace:klipper_keyboard_highlight into master

Reason for the change

Klipper's new highlight handling shows the highlight if the delegate is hovered. The ListView's currentItem is updated as hovered is changed, but not in the other direction, so the highlight will not correctly show on arrow key navigation, only mouse interaction.

Instead, show the highlight on being the currentItem. As the currentItem is updated on hover, this shows a highlight on either keyboard or mouse interaction, and there is no chance of a double highlight.

Test plan

  1. Open clipper
  2. Arrow key around
  3. (without) no highlight (with) highlight

Screenshots or screen recordings

N/A

Bugs fixed

BUG: 500055 FIXED-IN 6.3.1

Pinging @davidedmundson as you changed that recently - the issue is not in 6.3.0, but will be in 6.3.1 tomorrow

Merge request reports