<table><tr><td style="">ltoscano 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/D19557">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19557#428619" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19557#428619</a>, <a href="https://phabricator.kde.org/p/ognarb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ognarb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19557#428618" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19557#428618</a>, <a href="https://phabricator.kde.org/p/ltoscano/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ltoscano</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19557#428617" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19557#428617</a>, <a href="https://phabricator.kde.org/p/ognarb/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ognarb</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>They use the old link for the image, also the jpg version instead of the png. I tested with new page and old page and the result is visually the same.</p></div>
</blockquote>

<p>But do the existing pages use the old link for the image with the new CSS?</p></div>
</blockquote>

<p>Yes the link to the css didn't change, only the content. So even the old page links to the new CSS.</p></div>
</blockquote>

<p>Ok, so let's back one minute, and maybe it was the case in the original review, despite what myself and other people suggested, what if:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">the changes to the XSL files are reverted</li>
<li class="remarkup-list-item">top-kde.jpg is rewritten using the same size of the original, and of course same format. No need to add the new kde.png. You may want to add it, but same format, so that we can switch in KF6.</li>
</ul>

<p>This should ensure no unexpected changes if any user expect the image to be that size, and both newly-generated pages and the old ones will have the same aspect.</p>

<p>Sorry if it was the case in the first review.</p>

<p>The point about the commit message is still valid.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R238 KDocTools</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19557">https://phabricator.kde.org/D19557</a></div></div><br /><div><strong>To: </strong>ognarb, VDG, Documentation, yurchor, ltoscano<br /><strong>Cc: </strong>yurchor, rooty, ltoscano, bruns, abetts, broulik, aacid, kde-frameworks-devel, kde-doc-english, gennad, michaelh, ngraham, skadinna<br /></div>