Working great
This can be useful for example to do window managment via Meta key. BUG:475876
Sorry that I have to give giving opposite arguments here. I still think minimum sizes make sense.
Ideally every plasmoid should aim for good responsiveness on different sizes, but there is a point after which many will break. Wrong layouts, texts overflowing, etc...
It might be a bit unfair to those widgets to get bug reports on sizes they're not supposed to work with.
A different case is whether those widgets are too conservative on those constraints.
Now, Plasma-Windowed has been relegated to "expert-use" only, and this may be a risk we can easily accept, so not blocking this MR anyway.
Looking good now! As long as we're still getting the max sizes the behavior is correct.
I see now that the problem was the indirect use of attached Layout properties. Thanks for fixing that ugly references.
The bindings are not invalid in principle. We need to set the SwipeView`s implicit size to the maximum of the individual content sizes to limit the implicit/minimum size of the whole applet.
Otherwise, by default the swipeview takes the content size of the item that it is currently selected, and the applet size will be jumping sizes between views.
The only case where they size is imposed externally and the implicit content wouldn't matter is when it lives in the system tray.
If this is triggering a bug, we can maybe workout a different solution that doesn't trigger it, but the iteration is required. It might be an initilization/destruction problem, as the different views can be loaded dynamically depending on the available weather data.
Ismael Asensio (25b31818) at 25 Mar 17:03
Instead of the previous default which used "Most recent entries",
Ideally "Rating" could be a better candidate, but it depends on how the actual rating is done.
Ismael Asensio (25b31818) at 25 Mar 17:03
Change default sorting of entries to "Most Downloads"
Instead of the previous default which used "Most recent entries",
Ideally "Rating" could be a better candidate, but it depends on how the actual rating is done.
Ismael Asensio (25b31818) at 25 Mar 16:26
Change default sorting of entries to "Most Downloads"
Ismael Asensio (e7606c6c) at 21 Mar 18:53
Current TapHandler toggles the expanded/compressed status of the delegates. This has two problems:
Instead, just use it to expand the delegate and set focus to it.
Ismael Asensio (e7606c6c) at 21 Mar 18:53
kcms/notifications: Improve the click interaction for events
Failing test is the clang-format test on unrelated code. Merging...
Ismael Asensio (e7606c6c) at 21 Mar 18:27
kcms/notifications: Improve the click interaction for events
... and 7 more commits
About the "layout going south", now I can see the images.
The "sound" TextField was not south. It just have a minimum size of 5 gridUnits, which I think is enough for the minimum sizes in systemsettings or kcmshell. We can even eliminate this condition, but as part of another MR.
For the new "execute" TextField I forgot to add a line to the suggestion, so the Layout does not react as it should:
Layout.fillWidth: true
Layout.preferredWidth: implicitWidth + 3 * (Kirigami.Units.smallSpacing + Kirigami.Units.iconSizes.medium)
I just saw I forgot this missing line to make the textfield able to reduce and not impose its preferredWith to the Layout.
Sorry I changed a nearby line, so the git algorithm is totally clueless on how to rebase now.