Page MenuHome

Adjust tooltips display
AbandonedPublic

Authored by Bastien Montagne (mont29) on Aug 9 2014, 12:20 PM.

Details

Summary

Updated patch for T24055 against current master.

Diff Detail

Event Timeline

Shane Ambler (sambler) retitled this revision from to Adjust tooltips display.Aug 9 2014, 12:20 PM
Shane Ambler (sambler) updated this object.
Shane Ambler (sambler) set the repository for this revision to rB Blender.
Shane Ambler (sambler) added a subscriber: BF Blender.
Shane Ambler (sambler) updated this revision to Diff 2394.

@Shane Ambler (sambler), don't add BF Blender to the subscriber... we all get emails then...

Bastien Montagne (mont29) requested changes to this revision.

Patch looks nice and simple, but should not use U for temp data. I would rather suggest to use e.g. uiBut->flag (see interface_intern.h) for that.

This revision now requires changes to proceed.Aug 9 2014, 3:44 PM
Shane Ambler (sambler) updated this revision to Diff 2399.

Use uiBut->flag for temporary storage instead of UserDef->flag

Jonathan Williamson (carter2422) requested changes to this revision.

Hey @Shane Ambler (sambler), nice change. It seems to work perfectly.

I just have one thing to note.

I'd like to see you update the tooltip for the Tooltips setting in User Preference. Otherwise there is no way for users to know that holding alt will force show tips. Something like this should work well:

Display Tooltips; if off, hold Alt to force show tooltip

This revision now requires changes to proceed.Aug 9 2014, 8:38 PM
Shane Ambler (sambler) updated this revision to Diff 2401.

Extend tooltip for tooltip preference setting to expose the alt/option alternative.

@Jonathan Williamson (carter2422) I had thought of explaining that on the wiki page but it is still shorter than some tooltips so is a good workable idea.

Bastien Montagne (mont29) accepted this revision.

Lgtm, will commit soon.

Bastien Montagne (mont29) commandeered this revision.
Bastien Montagne (mont29) abandoned this revision.

Committed as rB64c7b2a122d6099, thanks! :)