<table><tr><td style="">rjvbb 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/D17054">View Revision</a></tr></table><br /><div><div><p>This change does more than just enabling hidpi support in the plist.</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>Given you've been doing the same patch for multipple apps now, any chance this can be generated using an ECM macro, pernhaps populated from the app's desktop file or appstream data?</p></blockquote>

<p>Generating the correct XML from that data is maybe a bit of a tall order for cmake's language?</p>

<p>As to the hidpi support: why is it necessary to set that in the plist in addition to what's done in the code? This seems like something that should be fixed in Qt's cmake modules or even in cmake itself (wherever the default Info.plist model comes from).</p>

<p>FWIW, <tt style="background: #ebebeb; font-size: 13px;">LSMultipleInstancesProhibited</tt> should of course not be set by default (should it for Okular which seems to launch in multiple instances on Linux?)</p>

<p>Lastly: please add <tt style="background: #ebebeb; font-size: 13px;">kde-mac</tt> to the subscriber list for Mac-related reviews.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17054">https://phabricator.kde.org/D17054</a></div></div><br /><div><strong>To: </strong>darcyshen, Okular<br /><strong>Cc: </strong>rjvbb, broulik, ltoscano, okular-devel, ngraham, darcyshen, aacid<br /></div>