Add tooltip to configure panel button
ClosedPublic

Authored by GB_2 on Jan 19 2019, 12:02 PM.

Details

Summary

Adds a tooltip to the configure panel button.

Test Plan

Hover over the configure panel button

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
GB_2 created this revision.Jan 19 2019, 12:02 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 19 2019, 12:02 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
GB_2 requested review of this revision.Jan 19 2019, 12:02 PM
ndavis accepted this revision.Jan 19 2019, 12:08 PM
ndavis added a subscriber: ndavis.

I think this is a good idea. We've always done a rather poor job of telling new users how to modify the panel and this helps a bit. Now let's wait for input from Plasma people.

This revision is now accepted and ready to land.Jan 19 2019, 12:08 PM
ngraham accepted this revision.Jan 19 2019, 3:07 PM
ngraham added a subscriber: ngraham.

+1 from me too. Makes sense for this particular application, and as a general principle, I think it should be mandatory for all icons-only buttons to have a tooltip so there's at least some explanatory text somewhere.

why all tooltips are so big
in windows and in gnome
they are smaller and still readable
what do u think

Not all tooltips are this big, this is just the special PlasmaCore.ToolTipArea Tooltip.

It does kind of bug me how different these types of tooltips look from the "standard" kind. They're much bigger and their backgrounds are light instead of dark. I wouldn't mind seeing their appearance tweaked to better match other tooltips.

Material for another patch though. :)

This revision was automatically updated to reflect the committed changes.