<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/D26903">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/D26903#600771" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D26903#600771</a>, <a href="https://phabricator.kde.org/p/cullmann/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@cullmann</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Would something like this be ok?</p>

<p>Not sure what to write about sizes, e.g. for the program icon we don't mention anything either.</p></div>
</blockquote>

<p>The program icon I have and such would assume others as well to be thought to have to satisfy the usual icon sizes (as defined for application cons in the xdg spec), given it is named "icon" after all.</p>

<p>For the programLogo things are different, logo display sizes are not specified somewhere (unless one indirectly assumes them given that the application icon is named as default). So possibly best would be to hint that any pixmaps passed here (via direct or also via QIcon) should follow the xdg icon spec, so have some 48x48 icon (or multiple of).</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The QPixmap case was missing in the docs, too, btw.</p></blockquote>

<p>Good catch :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26903">https://phabricator.kde.org/D26903</a></div></div><br /><div><strong>To: </strong>cullmann, apol<br /><strong>Cc: </strong>kossebau, dhaumann, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>