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

Reply via email to