<div>davidedmundson added inline comments.</div><br /><div><strong>INLINE COMMENTS</strong><div><div>desktoppackage/contents/views/Panel.qml:120 >preferredWidth changes, tries to update the length, but it can't because minimum/maximum don't allow<br />
<br />
the length will update, the size of the window won't.<br />
<br />
everything in the UI should be based around it's actual width not it's preferredWidth. <br />
<br />
>minimum/maximum change, but preferredWidth doesn't<br />
<br />
It will resize to  qBound(min, preferred, max); <br />
so it might change, it might not.<br />
<br />
>does the panel size get updated correctly?<br />
<br />
I /think/ so, would be great if you could check too.<br />
I can push to a branch if you want to test (or I think you can get them with arc somehow)<br />
</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1162" rel="noreferrer">https://phabricator.kde.org/D1162</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma, mart<br /><strong>Cc: </strong>mart, plasma-devel<br /></div>