<table><tr><td style="">kossebau added a comment.
</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/D19787">View Revision</a></tr></table><br /><div><div><p>After a first coffee, I realize this very logic actually already does upsampling all the time, given it takes the floor of the nodesize/samplesize expression, not the ceiling. And with eyes made sensitive, I see this now all the time.</p>

<p>Rereading the CSS3 spec, it seems they actually want the proposed <tt style="background: #ebebeb; font-size: 13px;">qRound</tt> (+ excluding 0 as nearest integer), so the actual proper algorithm to meet the "round" value of "border-image-repeat" would be:</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);">textureRect.setWidth(qMax(1, qRound((qreal)nodeRect.width() / m_elementNativeSize.width())));</pre></div>

<p>So we still have upsampling, at least with the pixmap material provided by Fluffy Bunny, which seems 1:1 pixels, so not a happy camper yet here for my mission to actually bring Fluffy Bunny back:<br />
<a href="https://phabricator.kde.org/F6695912" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6695912: Screenshot_20190316_075738.png</a></p>

<p>Then, looking closer at screenshots of Fluffy Bunny from Plasma4 times, I cannot see any scaling/stretching at all:<br />
<a href="https://frinring.files.wordpress.com/2009/01/x-bar.png" class="remarkup-link" target="_blank" rel="noreferrer">https://frinring.files.wordpress.com/2009/01/x-bar.png</a><br />
<a href="https://liquidat.files.wordpress.com/2008/06/fluffybunny.png" class="remarkup-link" target="_blank" rel="noreferrer">https://liquidat.files.wordpress.com/2008/06/fluffybunny.png</a></p>

<p>Other tiling based themes indicate that indeed no stretching was done:<br />
<a href="https://liquidat.files.wordpress.com/2008/06/slickback.png" class="remarkup-link" target="_blank" rel="noreferrer">https://liquidat.files.wordpress.com/2008/06/slickback.png</a></p>

<p>So... with all that in mind and yet another coffee, IMHO we should restore the logic of Plasma4 then. And if somebody wants the "round" repeat style, we should introduce another hint for them. <br />
Returning to the initial logic, things look like this, which I favour over the other:<br />
<a href="https://phabricator.kde.org/F6695944" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6695944: Screenshot_20190316_084031.png</a><br />
Updating patch next.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19787">https://phabricator.kde.org/D19787</a></div></div><br /><div><strong>To: </strong>kossebau, Plasma, mart<br /><strong>Cc: </strong>apol, sitter, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>