<table><tr><td style="">rjvbb requested changes to this revision.<br />rjvbb added a comment.<br />This revision now requires changes to proceed.
</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>Actually, I realise this patch too is a somewhat stripped down version of a patch I've been using for a long time in my MacPorts packaging for Okular:</p>

<p><a href="https://github.com/RJVB/macstrop/blob/master/kf5/kf5-okular/files/patch-open-docs-from-Finder.diff" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/RJVB/macstrop/blob/master/kf5/kf5-okular/files/patch-open-docs-from-Finder.diff</a></p>

<p>This diff is simply named wrong. Hidpi (retina) support is only one of the purposes of this custom Info.plist . In addition to that it also adds support for configuring the copyright message, adds the indicator that Carbon is required (mostly for good measure) and above all, registers what kind of documents the application can handle with LaunchServices (read: the Finder). That was the initial purpose of using a custom Info.plist, the other changes are just icing on the cake. FWIW, that <tt style="background: #ebebeb; font-size: 13px;">CFBundleDocumentTypes</tt> is pointless without a change to the code, handling <tt style="background: #ebebeb; font-size: 13px;">QEvent::FileOpen</tt> (see the shell.cpp diff in the linked patch).</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, rjvbb<br /><strong>Cc: </strong>rjvbb, broulik, ltoscano, okular-devel, ngraham, darcyshen, aacid<br /></div>