Review Request 113101: Fix Bug 287983 - Dolphin truncates tooltip information for long file names

Kai Uwe Broulik kde at privat.broulik.de
Thu Oct 10 14:09:24 BST 2013



> On Oct. 10, 2013, 1:08 p.m., Kai Uwe Broulik wrote:
> >

Can you test it doesn't break with huge font sizes? A fixed 250px maximum width looks dangerous to me.


- Kai Uwe


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113101/#review41505
-----------------------------------------------------------


On Oct. 4, 2013, 5:53 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113101/
> -----------------------------------------------------------
> 
> Review request for Dolphin.
> 
> 
> Bugs: 287983
>     http://bugs.kde.org/show_bug.cgi?id=287983
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Use KStringHandler and QTextLayout to wrap the text (file name)
> into the maximum width of the label "name".
> 
> 
> Diffs
> -----
> 
>   dolphin/src/views/tooltips/filemetadatatooltip.cpp c22f6be 
> 
> Diff: http://git.reviewboard.kde.org/r/113101/diff/
> 
> 
> Testing
> -------
> 
> Works
> 
> 
> File Attachments
> ----------------
> 
> Tooltip Widget with a long file name - Dolphin 4.11.2
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/10/04/b5e3213b-ca9b-4abc-b2b7-1f62f8748521__snapshot11.png
> Tooltip Widget with a long file name - Dolphin 4.11.3 (Patched)
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/10/04/26a7063d-ad96-4a8d-a0b2-fc0414ab1304__snapshot12.png
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20131010/f45adb79/attachment.htm>


More information about the kfm-devel mailing list