<table><tr><td style="">ngraham 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/D15580">View Revision</a></tr></table><br /><div><div><p>This is super fantastic.</p>

<p>I have a few more observations from use:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When using one of the quick annotations, the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Quick Annotations</span></span></span> button should have a checked state so you can tell that one of its tools is active (otherwise the previously-active toolbar button still looks checked despite not being active)</li>
<li class="remarkup-list-item">Maybe add a new menu item at the bottom of the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Quick Annotations</span></span></span> menu saying "Show all" that will open the full toolbar</li>
<li class="remarkup-list-item">The menu item that says, "Configure annotations..." should probably say "Configure quick annotations..." since it applies to the annotations in the quick annotations list, not all annotations more generally. Or even just "Configure..."</li>
<li class="remarkup-list-item">I would add text to the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Keep the active annotation button active after use</span></span></span> toolbutton, since otherwise it's a bit hard to tell what it does. Maybe "Keep active" or "Keep active after use" (maybe that's too long)?</li>
<li class="remarkup-list-item">Maybe don't have the Keep Active button checked by default. In testing, it feels more natural to have to click on an annotation's button after each use.</li>
<li class="remarkup-list-item">When an existing annotation is selected, it would be really nice if the controls for choosing the color, line thickness, opacity, font details etc. became active again and allowed you to change that annotation's appearance after the fact. You can already edit it by right-clicking and going to <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Properties</span></span></span>, but it would be even nicer to be able to do this in a more direct manner IMO</li>
<li class="remarkup-list-item">It would be nice if highlight, underline, squiggle, and strikethrough annotations were mouse-selectable while the annotations toolbar is open. That way it would be more obvious how to delete them, and you could change their properties using the above method</li>
<li class="remarkup-list-item">It might be nice if hitting the <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">9</kbd> key multiple times cycled through the items in the shape annotation menu</li>
<li class="remarkup-list-item">The <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">0</kbd> key could do the same for the stamp annotation</li>
<li class="remarkup-list-item">Now that we have a way to add stamp annotations using this new method, we need to show the message that stamp annotations are an experimental feature so that users know that they can't necessarily rely on it. Currently they only see this in the annotation settings window</li>
</ul>

<p>Overall this is feeling really good and I think it's quite close to being ready for prime time.</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/D15580">https://phabricator.kde.org/D15580</a></div></div><br /><div><strong>To: </strong>simgunz, Okular, VDG<br /><strong>Cc: </strong>trickyricky26, simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, johnzh, andisa, siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen<br /></div>