Review Request 125200: Don't overflow action buttons in notification

[Martin Klapetek]

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


The text is elided in the Plasma button style, but it has to do with the width computation in the style, I'm not sure exactly, but my guess is line 115, the "Math.max(theme.mSize(theme.defaultFont).width*12, style.minimumWidth);".

So does this fix the linked bug when you use Arial font?

- Martin Klapetek


On Sept. 12, 2015, 8:10 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125200/
> -----------------------------------------------------------
> 
> (Updated Sept. 12, 2015, 8:10 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: 352125
>     https://bugs.kde.org/show_bug.cgi?id=352125
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Somehow the maximumWidth constraint is not propagated through the Repeater.
> 
> 
> Diffs
> -----
> 
>   applets/notifications/package/contents/ui/NotificationItem.qml 350faa8 
> 
> Diff: https://git.reviewboard.kde.org/r/125200/diff/
> 
> 
> Testing
> -------
> 
> Buttons now respect the maximumWidth
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150912/0e88c162/attachment.html>