Revert "Revert "Reduce the indicator arrow size for press-and-hold menus in QToolButtons""
ClosedPublic

Authored by ndavis on Sat, Dec 21, 9:44 AM.

Details

Summary

This reverts commit c9aa535863e09bf736b1fb64c182b93530780c5d.

Also makes the small down arrow pixel perfect. I will make the other arrows pixel perfect in a separate patch.

After D19890 (27bcd1be9c2c02067bf15c62fdb7f7e2ad394bd9) was reopened, there was some discussion about the reason it was reverted and the problems it introduced. It is possible that an icon could partially cover this arrow, but it seemed uncommon enough that ignoring the problem was deemed acceptable. The type of control that this arrow is used for is also pretty uncommon because it does not feel very good to use with a mouse or touchpad.

Diff Detail

Repository
R31 Breeze
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ndavis created this revision.Sat, Dec 21, 9:44 AM
Restricted Application added a project: Plasma. · View Herald TranscriptSat, Dec 21, 9:44 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ndavis requested review of this revision.Sat, Dec 21, 9:44 AM
This comment was removed by ndavis.
ndavis edited the summary of this revision. (Show Details)Sat, Dec 21, 9:55 AM
ngraham accepted this revision.Sat, Dec 21, 4:40 PM
This revision is now accepted and ready to land.Sat, Dec 21, 4:40 PM
ndavis edited the summary of this revision. (Show Details)Sat, Dec 21, 5:48 PM
This revision was automatically updated to reflect the committed changes.