<table><tr><td style="">Zren created this revision.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6322" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Wait until the ScrollView component has completed before attempting to<br />
dynamically size the icons based on the viewport width/height.</p>

<p>A larger write up is in <a href="https://phabricator.kde.org/D6188" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D6188</a>, but essentially, variables are recalculated like this before the panels are added:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">qml: cellHeight iconHeight 108
qml: onCellHeightChanged 108
qml: cellWidth iconWidth 92
qml: onCellWidthChanged 92
qml: cellHeight iconHeight 108
qml: cellHeight iconHeight 108
qml: cellWidth iconWidth 92
qml: scrollArea.onCompleted
qml: cellWidth iconWidth 92
qml: cellHeight iconHeight 108
qml: cellHeight iconHeight 108
qml: cellWidth iconWidth 92
qml:     availableColumns 20.869565217391305  = containerSize / cellSize 1920 92
qml:     availableColumns 20  floored
qml:     allColumnSize 1840
qml:     extraSpace 80
qml:     extraSpacing 4
qml: cellWidth extraWidth 4
qml: onCellWidthChanged 96
qml: cellHeight iconHeight 108
qml:     availableColumns 10  = containerSize / cellSize 1080 108
qml:     availableColumns 10  floored
qml:     allColumnSize 1080
qml:     extraSpace 0
qml:     extraSpacing 0                                                                                                                                                                                                                                                        
qml: cellHeight extraHeight 0</pre></div>

<p>After the panels are added, it will do:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"
file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"
qml: cellHeight iconHeight 108
qml:     availableColumns 9.99074074074074  = containerSize / cellSize 1079 108
qml:     availableColumns 9  floored
qml:     allColumnSize 972
qml:     extraSpace 107
qml:     extraSpacing 11.88888888888889
qml: cellHeight extraHeight 11.88888888888889
qml: onCellHeightChanged 119.88888888888889
file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"
file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"
qml: timeLeft 0 1498071212427
qml: cellHeight iconHeight 108
qml:     availableColumns 9.722222222222221  = containerSize / cellSize 1050 108
qml:     availableColumns 9  floored
qml:     allColumnSize 972
qml:     extraSpace 78
qml:     extraSpacing 8.666666666666666
qml: cellHeight extraHeight 8.666666666666666
qml: onCellHeightChanged 116.66666666666667
file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"
file:///home/chris/.local/share/plasma/plasmoids/org.kde.desktopcontainment/contents/ui/FolderItemDelegate.qml:282:17: QML Text: Binding loop detected for property "width"</pre></div>

<p>It may sometimes skip the <tt style="background: #ebebeb; font-size: 13px;">plasmoid.screenGeomertry.height = 1079</tt> and only calculate it at 1050px.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6322" rel="noreferrer">https://phabricator.kde.org/D6322</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderView.qml</div></div></div><br /><div><strong>To: </strong>Zren, hein<br /><strong>Cc: </strong>plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>