<table><tr><td style="">rjvbb accepted this revision.<br />rjvbb added a comment.<br />This revision is now accepted and ready to land.
</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/D10808" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I can hardly NOT accept this, given I'm the original author :)</p>

<p>Adding document-type icons isn't strictly necessary. The Finder will display documents with the icon of the default application configured to open the type in question, or with the application configured for the individual document. Applications that provide icons for the different document types they support usually do this by adapting the application icon instead of using a generic icon for the document type (exceptions aside of course). (I'm not at my Mac right now and hardly use the Finder in practice; I wouldn't be surprised if the Finder actually tagged application icon with a standard document type representation when apps don't provide their own icons.)<br />
That means it wouldn't be appropriate to bundle the document-type icons from a standard Freedesktop.org theme (let alone from Breeze, IMHO)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10808" rel="noreferrer">https://phabricator.kde.org/D10808</a></div></div><br /><div><strong>To: </strong>sbragin, Okular, rjvbb<br /><strong>Cc: </strong>rjvbb, Okular, michaelweghorn, ngraham, aacid<br /></div>