Review Request 128429: Fix strings in tooltips

Chusslove Illich caslav.ilic at gmx.net
Sun Jul 24 19:42:51 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128429/#review97805
-----------------------------------------------------------



When assembling tooltips with +=, every xi18nc call will return strings with "<html>...</html>", so the final tooltip will not be exactly normal rich text markup. But Qt's rich text engine is forgiving of that, so practically shouldn't be a problem.

To have clean rich text markup, one would have to conditionally produce paragraphs as KLocalizedString (with kxi18nc), store them in a list or something, and at the end based on the number of pieces create a meta-string of the type kxi18nc("@info:tooltip", "%1%2%3..."), add all the pieces with multiple .subs(), and call .toString().


processui/ProcessModel.cpp (line 1573)
<https://git.reviewboard.kde.org/r/128429/#comment65914>

    Missing closing tags.


- Chusslove Illich


On Јул 18, 2016, 7:38 по п., Martin Tobias Holmedahl Sandsmark wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128429/
> -----------------------------------------------------------
> 
> (Updated Јул 18, 2016, 7:38 по п.)
> 
> 
> Review request for Plasma, Albert Astals Cid, Chusslove Illich, and John Tapsell.
> 
> 
> Repository: libksysguard
> 
> 
> Description
> -------
> 
> Changed the strings to use proper kuit markup.
> 
> 
> Diffs
> -----
> 
>   processui/ProcessModel.cpp c55cea6 
> 
> Diff: https://git.reviewboard.kde.org/r/128429/diff/
> 
> 
> Testing
> -------
> 
> Viewed all the tooltips.
> 
> 
> Thanks,
> 
> Martin Tobias Holmedahl Sandsmark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160724/7c003174/attachment.html>


More information about the Plasma-devel mailing list