Bug 472811 - Items' background frames highlight on hover even when disabled
Summary: Items' background frames highlight on hover even when disabled
Status: RESOLVED FIXED
Alias: None
Product: libplasma
Classification: Frameworks and Libraries
Component: components (other bugs)
Version First Reported In: 6.1
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-07-30 20:57 UTC by Nate Graham
Modified: 2025-02-26 16:49 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 6.3.2
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2023-07-30 20:57:12 UTC
STEPS TO REPRODUCE
1. Have only one screen
2. Apply https://invent.kde.org/plasma/kscreen/-/merge_requests/227
3. Open KScreen applet and hover over one of the disabled buttons

OBSERVED RESULT
Button's icon becomes active

EXPECTED RESULT
Nothing happens
Comment 1 Nate Graham 2023-07-30 20:57:46 UTC
Dummy comment to prevent the bot from marking this as a Qt6 issue (it's pre-existing).
Comment 2 Nate Graham 2025-02-22 20:08:10 UTC
Looks like it's actually PlasmaComponents3.Button that's doing this.

qqc2-desktop-style doesn't have the same issue; it's just here.
Comment 3 Nate Graham 2025-02-22 20:14:00 UTC
Looks like it affects everything.
Comment 4 Bug Janitor Service 2025-02-22 20:42:56 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libplasma/-/merge_requests/1276
Comment 5 Nate Graham 2025-02-26 16:49:55 UTC
Git commit 1d84f62d063bd0890f2108559e3e92aa8c1e7497 by Nate Graham.
Committed on 26/02/2025 at 16:47.
Pushed by ngraham into branch 'master'.

Don't highlight disabled items on hover

In the past we didn't have to do this because we simply bound
`hoverEnabled` to `enabled`. But then we moved away from that to make
tooltips for disabled items appear on hover. Thus, we need to check
for enablement when deciding when to use the hover state for hovered
items.
FIXED-IN: 6.3.2

M  +2    -2    src/declarativeimports/plasmacomponents3/ComboBox.qml
M  +1    -1    src/declarativeimports/plasmacomponents3/Dial.qml
M  +1    -1    src/declarativeimports/plasmacomponents3/MenuItem.qml
M  +3    -3    src/declarativeimports/plasmacomponents3/RadioIndicator.qml
M  +2    -2    src/declarativeimports/plasmacomponents3/RangeSlider.qml
M  +4    -4    src/declarativeimports/plasmacomponents3/RoundButton.qml
M  +2    -2    src/declarativeimports/plasmacomponents3/ScrollBar.qml
M  +2    -2    src/declarativeimports/plasmacomponents3/Slider.qml
M  +3    -3    src/declarativeimports/plasmacomponents3/SpinBox.qml
M  +1    -1    src/declarativeimports/plasmacomponents3/SwitchIndicator.qml
M  +1    -1    src/declarativeimports/plasmacomponents3/TextArea.qml
M  +2    -2    src/declarativeimports/plasmacomponents3/TextField.qml
M  +2    -2    src/declarativeimports/plasmacomponents3/private/ButtonBackground.qml
M  +5    -0    tests/components/combobox3.qml
M  +13   -0    tests/components/menu.qml
M  +21   -2    tests/components/roundbutton3.qml
M  +20   -0    tests/components/slider3.qml
M  +6    -0    tests/components/textarea3.qml
M  +5    -0    tests/components/textfield3.qml

https://invent.kde.org/plasma/libplasma/-/commit/1d84f62d063bd0890f2108559e3e92aa8c1e7497