[effects/startupfeedback] Scale application icon size with cursor size
ClosedPublic

Authored by broulik on Wed, Oct 16, 9:12 PM.

Details

Summary

KWin doesn't use Qt scaling and as such the icon remains tiny.
Instead, make it follow the cursor size as the user is likely to have larger cursors on high dpi.

Test Plan
  • I can actually see my bouncy cursor now
  • also works with blinking

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Wed, Oct 16, 9:12 PM
Restricted Application added a project: KWin. · View Herald TranscriptWed, Oct 16, 9:12 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
broulik requested review of this revision.Wed, Oct 16, 9:12 PM
broulik edited reviewers, added: KWin; removed: kwinslet.
zzag added a subscriber: zzag.Thu, Oct 17, 7:42 AM
zzag added inline comments.
effects/startupfeedback/startupfeedback.cpp
287

What's this 1.5 factor?

broulik added inline comments.Thu, Oct 17, 7:42 AM
effects/startupfeedback/startupfeedback.cpp
287

Normal cursor size is 24, normal small icon size is 16.

zzag accepted this revision.Thu, Oct 17, 8:00 AM
This revision is now accepted and ready to land.Thu, Oct 17, 8:00 AM
This revision was automatically updated to reflect the committed changes.