<table><tr><td style="">cullmann 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><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>

<p>The QPixmap case was missing in the docs, too, btw.</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="diff" 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);"><span style="color: #000080">diff --git a/src/lib/kaboutdata.h b/src/lib/kaboutdata.h</span>
<span style="color: #000080">index 7b317f7..c9f8da0 100644</span>
<span style="color: #a00000">--- a/src/lib/kaboutdata.h</span>
<span style="color: #00a000">+++ b/src/lib/kaboutdata.h</span>
<span style="color: #800080">@@ -750,8 +750,10 @@ public:</span>
      * Use this if you need to have an application logo
      * in AboutData other than the application icon.
      *
<span style="color: #a00000">-     * Because KAboutData is a core class it cannot use QImage directly,</span>
<span style="color: #a00000">-     * so this is a QVariant that should contain a QImage.</span>
<span style="color: #00a000">+     * Because KAboutData is a core class it cannot use QImage/QPixmap/QIcon directly,</span>
<span style="color: #00a000">+     * so this is a QVariant that should contain a QImage/QPixmap/QIcon.</span>
<span style="color: #00a000">+     *</span>
<span style="color: #00a000">+     * QIcon should be preferred, to be able to properly handle HiDPI scaling.</span>
      *
      * @param image logo image.
      * @see programLogo()
<span style="color: #800080">@@ -964,8 +966,8 @@ public:</span>
     /**
      * Returns the program logo image.
      *
<span style="color: #a00000">-     * Because KAboutData is a core class it cannot use QImage directly,</span>
<span style="color: #a00000">-     * so this is a QVariant containing a QImage.</span>
<span style="color: #00a000">+     * Because KAboutData is a core class it cannot use QImage/QPixmap/QIcon directly,</span>
<span style="color: #00a000">+     * so this is a QVariant containing a QImage/QPixmap/QIcon.</span>
      *
      * @return the program logo data, or a null image if there is
      *         no custom application logo defined.</pre></div></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>