In D4204#80553, @davidedmundson wrote:

Though really any max > min on the client is undefined behaviour, so it's hard to say any is "right".

Fixed littke bug which would happend with code

plasmoid.Layout.minimumHeight = 150
plasmoid.Layout.maximumHeight = 130
plasmoid.Layout.maximumHeight = 150

Bug led to maximumHeight = 150 when minimumHeight would still be = 130

Removed Bindings

In D4204#80142, @mart wrote:

what i would like to have logically splitted is the management of the floating property (and having positionItem()/releasePosition used around)

and on the other hand the signal handlers of minimumWidthChanged/widthChanged etc in Appletappearance, that's the biggest part

In D4204#80106, @mart wrote:

can this be splitted in multiple reviews/commits?

Rewrited some code to better understanding
Fixed background-handle animation in plasmoidBackground. It was a little bug, but I don't see any sense to report it now when patch are ready. Fixed it by changing it to animate only handle-width instead of animation of full plasmoidBackground width.

Before anything else I'd like to explain why I need this patch. At winter holidays I had a free time and decided to rewrite plasma-simplemonitor(maded by dhabyx) - port it to plasma5 and improve it. As one of new features I added "skins" - posibility to change UI looks. Let's describe one skin as "row" and other as "column". Skins had different default sizes and different minimum sizes. As behaviour it sugges that when user change one skin to another plasmoid size supposed to change to default size of new skin. In plasma5 there is no resize function in plasmoid anymore. Instead docs pointed to use Layout properties. And so I tried to set maximum sizes in Layout. It doesn't work. When researching this problem it was occurred that there is no such functionality in plasmoids despite that plasmoid already uses Layout minimum sizes. And so I made this patch to fix that. When debugging this patch I found other bugs and fixed them alongside.

Fix commit message

Turn on background width animation again.

Rewrited commit message in according to kde commit polititcs

Summary: reworked set of patches to plasma-desktop/containments/desktop to fix bugs in plasmoid.
Fix for

• plasmoid applet missing maximum size handler
• plasmoid can't be set to declared minimumWidth
• plasmoid don't release space in layoutManager.properly due to size animations

In D4204#78701, @davidedmundson wrote:

That's quite a huge patch.
Can you give a much bigger explanation on exactly what it's doing and why.
"Fixes bug N" doesn't really explain what the original problem was.

It was not supposed to go there, but. sigh... arc...
I will try to make it better.