<table><tr><td style="">simgunz added a comment.
</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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D10859#214352" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D10859#214352</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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>.</p></div>
</blockquote>

<p>It is also possible to right click on the annotations bar to access directly the annotations configuration. My original idea was to change the behaviour in a way that right clicking on a tool on the bar allows accessing the config of that tool directly in a popup window (faster than  <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>. This would require small changes I think.</p>

<p>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>This solution is probably better but requires a complete redesign. Let's continue the discussion on the Task.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

<p>Strange. I did not touch at all that part of the code.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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></blockquote>

</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>