D9973: ktooltipwidget: Fix tooltip positioning

Michael Heidelbach noreply at phabricator.kde.org
Sat Jan 20 06:40:42 GMT 2018


michaelh added inline comments.

INLINE COMMENTS

> cfeck wrote in ktooltipwidget.cpp:155
> I don't understand. Either + or - is right here, but not both. Are you saying that you don't want to correct it at the second place, just because tests did not reveal a bug?

> Are you saying that you don't want to correct it at the second place, just because tests did not reveal a bug?

Yes, but it is rather like this: Due to my lack of my experience I don't introduce chances which I can't test or see the results of. More a question of 'dare' than 'want' :)

As margin is not included in sizeHint() I think it is correct to do

  +margin when tooltip is displayed right of rect and 
  -margin when displayed left of rect. 

Both would result in a margin-wide space between rect and tooltip which is necessary to dismiss the tooltip. 
@elvisangelaccio: What do you think?

REPOSITORY
  R236 KWidgetsAddons

REVISION DETAIL
  https://phabricator.kde.org/D9973

To: michaelh, elvisangelaccio, #frameworks, #dolphin, ngraham
Cc: cfeck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180120/166e890a/attachment.htm>


More information about the kfm-devel mailing list