-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119229/
-----------------------------------------------------------
Review request for Plasma.
Bugs: 336209
https://bugs.kde.org/show_bug.cgi?id=336209
Repository: plasma-desktop
Description
-------
Adapt iconsize to toolbox size introduces a blatant binding loop
It was changed to be 0.8 of the frame, but the frame was based on the icon size.
(line 78)
The binding loop was then "fixed" by wrapping in an item which is fixing
something that was wrong in the first place rather than the root problem.
The iconSize comes from units.iconSize.Small in ToolBoxRoot in the first place,
making the entire change pointless as it's
If we do want to change the icon we just need to change that one line in
ToolBoxRoot.
We certainly shouldn't be making changes that leave stray unused properties
about. in ToolBoxRoot.
Revert "remove undetected binding loop"
This reverts commit eddb56e5a7cac3cd631f1cb8e315af02f57779ee.
Revert "use standard icons sizes for the toolbox icon"
This reverts commit 216ee387648f7105561dabcf33d06ec880be65a3.
Revert "Adapt icon size to toolbox size"
Diffs
-----
toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml
7923b711d28417dde2c48ef7acf42880c1ac9d3a
Diff: https://git.reviewboard.kde.org/r/119229/diff/
Testing
-------
Thanks,
David Edmundson
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
