D8991: Use kSqueezedTextLabel for the label text on the statusbar

[Ahmad Samir]

ahmadsamir marked 2 inline comments as done.
ahmadsamir added a comment.


  I don't understand how phabricator works with inline comments. So just in case my reply to the tooltip part didn't get submitted:
  
  KSqueezedTextLabel does that by default; I just checked my local build, the tooltip is there when the text is squeezed on the statusbar. (And just to be sure https://api.kde.org/frameworks/kwidgetsaddons/html/ksqueezedtextlabel_8cpp_source.html#l00113).

INLINE COMMENTS

> elvisangelaccio wrote in dolphinstatusbar.cpp:330-334
> Please don't remove the tooltip, we can still show it if `m_label->isSqueezed()` is true.

KSqueezedTextLabel does that by default; I just checked my local build, the tooltip is there when the text is squeezed on the statusbar. (And just to be sure https://api.kde.org/frameworks/kwidgetsaddons/html/ksqueezedtextlabel_8cpp_source.html#l00113).

Thanks :).

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

To: ahmadsamir, elvisangelaccio
Cc: elvisangelaccio, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171125/91e715c3/attachment.htm>