<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham 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/D21910">View Revision</a></tr></table><br /><div><div><p>While this sort of works, now it makes the dialog way too tall, and I don't think it's the right approach to the problem. We should figure out what's causing the default size to be wrong and fix that, rather than leaving it broken and working around it in the proposed way.</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/D21910">https://phabricator.kde.org/D21910</a></div></div><br /><div><strong>To: </strong>mayankchoudhary, ngraham<br /><strong>Cc: </strong>ngraham, okular-devel, fbampaloukas, joaonetto, tfella, darcyshen, aacid<br /></div>