Summary: | CompactRepresentation's MouseArea intercepts hover event, preventing icon from having a highlight effect | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Nate Graham <nate> |
Component: | Notifications | Assignee: | Kai Uwe Broulik <kde> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | kde, materka, plasma-bugs |
Priority: | NOR | Keywords: | junior-jobs |
Version: | master | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/-/commit/39f05d2d5b74057ed6d9a0a8a1022865ff5046e8 | Version Fixed In: | 6.0 |
Sentry Crash Report: | |||
Bug Depends on: | |||
Bug Blocks: | 400295 |
Description
Nate Graham
2020-01-15 20:24:29 UTC
It's not quite as described. There is an IconItem, but the parent MouseArea is intercepting the hover event and not propogating downwards. It also uses SVGItem for some graphics which can't glow I proposed an idea on how to glow SvgItem in Bug 416310. Git commit 39f05d2d5b74057ed6d9a0a8a1022865ff5046e8 by Nate Graham. Committed on 24/08/2023 at 20:13. Pushed by ngraham into branch 'master'. applets/{batterymonitor,notifications}: Show CompactRep hover highlights Battery Monitor and Notifications have custom CompactRepresentations, but were not conditionally setting the active: property on hover for their custom icon items, causing them to lack the hover highlight effect that all other System Tray icons have. Related: bug 416310 FIXED-IN: 6.0 M +3 -1 applets/batterymonitor/package/contents/ui/CompactRepresentation.qml M +4 -0 applets/notifications/package/contents/ui/CompactRepresentation.qml M +5 -0 components/workspace/BatteryIcon.qml https://invent.kde.org/plasma/plasma-workspace/-/commit/39f05d2d5b74057ed6d9a0a8a1022865ff5046e8 |