I'm getting a binding loop: <Unknown File>: QML QQuickLayoutAttached: Binding loop detected for property "minimumWidth"
I'm guessing that's Layout.minimumWidth inside the ToolButtonStyle IconItem

If the developer has these needs, why not port to the PC3 version, which has this feature already?

In D28606#642509, @ngraham wrote:

If the developer has these needs, why not port to the PC3 version, which has this feature already?

@broulik said we can't do that for some things like notifications and KRunner because of how the tooltips work.

Darn. Maybe we should fix the PC3 tooltips though? :)

Adding Marco as a reviewer since I know he just looooooooooooooooves ToolButton sizing. :)

In D28606#642514, @ngraham wrote:

Adding Marco as a reviewer since I know he just looooooooooooooooves ToolButton sizing. :)

This doesn't actually affect toolbutton sizing, just icon sizing. Although, if you make the icon size too big, it'll bleed out of the button.

In D28606#642513, @ngraham wrote:

Darn. Maybe we should fix the PC3 tooltips though? :)

Maybe, but I don't know what that would entail.
For more context: I asked Kai about porting notifications to PC3 so that I could fix the size of icons with larger fonts. He said he wouldn't accept patches that do that for notifications or KRunner because of the issues with PC3, so I figured it would be easier to just add icon sizes to PC2.

He said he wouldn't accept patches that do that for notifications or KRunner because of the issues with PC3

I doubt that's what was meant.

There have definitely been some bad patches that blindly ported to PC3 without adequate testing. That's highly problematic.

But use of PC3 with work put into fixing components of issues that arise should certainly be a target. I'll bring it up in the meeting.

I don't think we should put any new feature in plasmacomponents2. it will be removed in qt6 anyways and work should be more concentrated to a) port away from it altogether abd b) if the pc3 version doesn't work, put effort in fixing that