View Revision
mart added inline comments.

INLINE COMMENTS
View Inlinedavidedmundson wrote in Label.qml:27

It probably made sense when it was written, but your comments about layouts is outdated.

No manual changes are needed.

try:
https://paste.kde.org/pgi2ui6e3

If it was a problem we'd be seeing it all the time in the config modules.

It is true for manual positioning and anchors, but I think the vast majority of the time where we'd want to be aligned to a button we'd be in a layout with that button.

I'm sure there will be some changes needed, but it'll be few. (if I remove the line in my QQC1 label locally, I can't see any)

In constrast run this on your workspace folder:
find -name '*.qml' | xargs grep 'height:' | grep paintedHeight

and you'll see the number of times we have to reverse this logic.

so, GridLayout and RowLayout align correctly, while Row still doesn't.

so, ok for me to remove it, and require *layouts are used instead of simple positioners everywhere


REPOSITORY
R242 Plasma Framework (Library)

REVISION DETAIL
https://phabricator.kde.org/D4508

EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, Plasma
Cc: davidedmundson, broulik, plasma-devel, Frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol