<table><tr><td style="">aacid accepted this revision.<br />aacid 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/D17801">View Revision</a></tr></table><br /><div><div><p>I have not carefully reviewed it, but it seems to be clearly better than what we have, so i'd say go ahead and commit it (without the setPlaceHolder changes that don't belong here i guess)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>dev_docs (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17801">https://phabricator.kde.org/D17801</a></div></div><br /><div><strong>To: </strong>yurchor, Okular, aacid<br /><strong>Cc: </strong>aacid, ngraham, okular-devel, tfella, darcyshen<br /></div>