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

Review request for Plasma and Marco Martin.


Description
-------

When you open the Activity Manager, and click the "Configure" button, the 
stacked config page displays a misplaced iconButton (see figure on the left).
That's because, in ActivityDelegate, the implicitHeight of the component is 
forced to 20.
Funny observation: this weird behavior shows up only when Activity Manager is 
displayed horizontally ... Vertically, everything is just fine.
Anyway the "patch" is just an one-liner so it should be OK, I suppose.
The fixed component is shown in the screenshot on the right.


Diffs
-----

  plasma/desktop/shell/activitymanager/package/contents/ui/ActivityDelegate.qml 
4af21e7 

Diff: http://git.reviewboard.kde.org/r/106999/diff/


Testing
-------

Works perfectly, both with horizontal and vertical placement (see shots below).


Screenshots
-----------

Activity Manager, before
  http://git.reviewboard.kde.org/r/106999/s/790/
Activity Manager, after
  http://git.reviewboard.kde.org/r/106999/s/791/


Thanks,

Diego Casella

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to