Review Request 120389: Fix collapsing panelcontroller buttons when they are too wide

Kai Uwe Broulik kde at privat.broulik.de
Sat Sep 27 15:14:23 UTC 2014


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

Review request for Plasma.


Bugs: 338636
    https://bugs.kde.org/show_bug.cgi?id=338636


Repository: plasma-desktop


Description
-------

The reason why it was broken on my machine was that the placeholder label used a different translation domain than the actual buttons resulting in the placeholder having the much shorter english variants and thus it worked fine on other machines.

I now added three readonly properties with the button text for use in the placeholder and tooltips. Given that I just removed three broken translations and only moved strings around and other than that it's a straight-forward bugfix I guess it's for 5.1 too.


Diffs
-----

  desktoppackage/contents/configuration/panelconfiguration/ToolBar.qml 5716405 

Diff: https://git.reviewboard.kde.org/r/120389/diff/


Testing
-------

Now in a horizontal panel the buttons are properly collapsed, tooltip appears on hovering. In vertical panel buttons have text (although that one is massive in German and the button gets wider than the window but that's out of this patch's scope)


Thanks,

Kai Uwe Broulik

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


More information about the Plasma-devel mailing list