D22512: [Dolphin] Hide tooltip instantly on key press

[Elvis Angelaccio]

elvisangelaccio added inline comments.

INLINE COMMENTS

> elvisangelaccio wrote in dolphinview.cpp:1427-1435
> We can fix this issue by using a lamba ;)
> 
>   connect(m_container->horizontalScrollBar(), &QScrollBar::valueChanged, this, [=] {
>       hideToolTip();
>   });

Have you tried to use this lambda in order to bring back the approach of https://phabricator.kde.org/D22512?id=62021 ? (which I prefer because we avoid to duplicate ` DolphinView::hideToolTip()`)

REPOSITORY
  R318 Dolphin

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

To: pdabrowski, #dolphin, ngraham, elvisangelaccio
Cc: broulik, elvisangelaccio, kfm-devel, pdabrowski, aprcela, vmarinescu, fprice, fbampaloukas, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190803/5f744ad6/attachment.htm>