drosca added a comment.
The problem is that PanelView::length() is always set to Layout.preferredSize, which says nothing about the actual panel length. PanelView::length() seems to be used only for restoring size of panel, but in most (all?) cases it gets overridden by minimumLength/maximumLength, so the panels are restored to correct size. Another use is in geometryByDistance, which can use size(), but it won't fix the main issue. I don't think there is a binding loop, because it doesn't actually resize the panel, it just saves the value to config and rest of PanelView::setLength is no-op. That being said, this timer is being fired too often, probably on height/width changed would be enough (but that needs more testing, because of the userConfiguring conditional). REPOSITORY rPLASMADESKTOP Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D1066 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: drosca, Plasma, davidedmundson Cc: davidedmundson, apol, plasma-devel
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel