<table><tr><td style="">mart added inline comments.
</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/D15333">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15333#inline-85370">View Inline</a><span style="color: #4b4d51; font-weight: bold;">McPain</span> wrote in <span style="color: #4b4d51; font-weight: bold;">main.qml:111</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">Especially the spacer.width being divided by 2 in the second part of the statement.</p></blockquote>

<p style="padding: 0; margin: 8px;">Look at all the parentheses I wrote ;)<br />
I divide by two a *subtraction* of spacer.width from width.</p>

<p style="padding: 0; margin: 8px;">Former we calculated a button size to fit the whole widget vertically (if the layout is horizontal) and vice versa but what if we don't have enough space to fit the widget horizontally?</p>

<p style="padding: 0; margin: 8px;">We have a 100x100 field. Okay, let height = 90 (margins, etc)<br />
Then, first button 90x90, spacer 5x90 (still okay), and one more button 90x90 (oops, fail: the widget is 185x90)</p>

<p style="padding: 0; margin: 8px;">Now we check the second dimension and shrink the buttons when necessary.<br />
First button 45x45, spacer 5x45, second button 45x45. The widget if 95x45, and it fits now.</p>

<p style="padding: 0; margin: 8px;">So, we have to subtract spacer width from parent width and divide it by 2, because we have two buttons and check if it's smaller than height. If so, we fit in the width.</p>

<p style="padding: 0; margin: 8px;">Same thing when layout is vertical.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">can you put this in the commit description as well?</p></div></div></div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15333">https://phabricator.kde.org/D15333</a></div></div><br /><div><strong>To: </strong>McPain, Plasma, ngraham<br /><strong>Cc: </strong>mart, davidedmundson, cfeck, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>