<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/D10859" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>This change seems sensible on first glance, but will it be effective? IMHO there's more to improve with the current UI than just changing the layouts of the config dialogs. For example, should there <em>be</em> global config dialogs in the first place? Instead, Right now, the only way to change the settings of the annotation tool you're currently using is very cumbersome: navigate all the way to <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Configure Okular -</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Annotations -</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">some tool -</span></span><span style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Edit</span></span></span>. That's a real chore, and makes it much harder than it needs to be. I would advocate making these settings visible in the main UI when you start using an annotation tool. That way you can easily change the settings for the tool you're currently using without having to switch contexts or open a bunch of new windows. The UI pattern of having a tool's settings contextually visible is very common--perhaps even near-universal--for software that has configurable editing tools.</p>

<p>Also, this patch results in the (perhaps inadvertent) removal of the Custom Stamp's ability to use a user-specified image for the stamp.</p>

<p>I would advocate some more design work in the Phabricator tasks before we produce code, or else we're likely to do a lot of work that we re-do later.</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/D10859" rel="noreferrer">https://phabricator.kde.org/D10859</a></div></div><br /><div><strong>To: </strong>simgunz, Okular, VDG, ngraham<br /><strong>Cc: </strong>ngraham, michaelweghorn, aacid<br /></div>