<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/D15580">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/D15580#552362" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15580#552362</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;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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</li>
</ul></div>
</blockquote>
<p>When an annotation is selected its annotation toolbar button is checked and <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> is checked, exactly as in the current Okular version. <br />
When you select a quick annotation tool what happens is that the corresponding annotation action gets checked. In a sense a quick annotation is just an alias for an effective annotation with a specified set of settings (color, etc.). For this reason the quick annotation cannot have a checked state. <br />
This however has a quirkiness in the case the annotation toolbar is hidden. In that case we are in <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> and an annotation tool is active but we do not see the corresponding checked action in the hidden toolbar. This can be "solved" by showing the annotation toolbar when a quick annotation is selected, but I think it is going to become frustrating very soon.</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>(otherwise the previously-active toolbar button still looks checked despite not being active)</p></blockquote>
<p>This should not happen. Probably you are referring to <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> that doesn't get unchecked, but that is the intended behavior and match that of the current Okular version.<br />
What happens should be:</p>
<p>Example 1:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Text Selction</span></span></span> is checked</li>
<li class="remarkup-list-item">Select <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 style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Green Highlighter</span></span></span></li>
</ul>
<p>Results:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> action gets checked</li>
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Highlighter</span></span></span> action gets checked</li>
</ul>
<p>Example 1:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> is checked</li>
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Ellipse</span></span></span> is checked</li>
<li class="remarkup-list-item">Select <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 style="color: #92969D;"> → </span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Green Highlighter</span></span></span></li>
</ul>
<p>Results:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Browse Mode</span></span></span> remains checked</li>
<li class="remarkup-list-item"><span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Highlighter</span></span></span> action gets checked</li>
</ul>
<p>If we make the quick annotation checkable we have the following problem:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">The <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Quick Annotation</span></span></span> won't display a <tt style="background: #ebebeb; font-size: 13px;">edit-draw</tt> icon with the text "Quick Annotation" but it will display a specific action, i.e. "Green Highlighter" which might be confusing for the user, given that <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Quick Annotation</span></span></span> would be never displayed. Exactly what happens now for the Stamp annotation (in this case you see "Approved" by default).</li>
</ul>
<p>Bottom line: unless there is a clever way to make the actions checkable but display the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Quick Annotation</span></span></span> button, I think that the current implementation is the the less worse situation. Regarding browse mode, we can also uncheck it when an annotation is selected, if there are no side effects.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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 That's a good idea. It would be: <tt style="background: #ebebeb; font-size: 13px;">[ ] Show Annotation Toolbar</tt> (menu items need to start with action verbs). And then the menu item in the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Tools</span></span></span> menu would be the same. probably the same QAction would just used for both; then they would both keep track of state properly.</li>
</ul></blockquote>
<p>Yes, but I do not know how to do it. See <a href="https://phabricator.kde.org/D15580#544534" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D15580#544534</a> for the details of the problem.</p>
<p>Actually this exact action already exists in <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Settings</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 ">Toolbars Shown</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 Toolbar</span></span></span> but I do not know of to access it and if it available when we set up the actions.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>I thought about this, but I decided to leave "Configure Annotations" given that in the same KCM module the user can configure the "annotation identity", which is not related to "Quick Annotations" but to "Annotations" in general. <br />
Changing it to "Configure" won't work either. This is a standard action that will appear in the "Configure Toolbar" and "Configure Shortcuts" dialog. If you call it "Configure" it will become ambiguous for the user.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>Currently the action name and thus the text is "Pin". You mean I should change "Pin" to "Keep Active"?</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>I agree with <a href="https://phabricator.kde.org/p/davidhurka/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@davidhurka</a> here regarding having it on by default. I cannot figure out a use case where a user wants to add a single annotation or keep reselecting it each time.<br />
My typical use case is: I am reading a paper/book and I check the highlighter to highlight the text multiple times while I read, so I want it always active. The highlighter is a common use case, think at e-readers which have only highlighters and notes.<br />
Another one is to fill in multiple forms with the typewriter.<br />
What real-world use cases do you have in mind where you do not need it always active?</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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></blockquote>
<p>Where would you show it? In the action tooltip? e.g "Approved - Stamps are an experimental feature"</p>
<hr class="remarkup-hr" />
<p>The things from here and below are very interesting but also quite complicated to implement and time-consuming. I would implement them in following revisions, or we won't ever terminate this one while trying to make the toolbar perfect. Release early, release often.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>I agree</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>Well at this point even the other annotations, so you can change their properties.</p>
<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><ul class="remarkup-list">
<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>
</ul></blockquote>
<p>Rather difficult to do. It means that everytime the shortcut is triggered we need to reset the shortcuts of the actions in that list. This should be implemented upstream in KSelectAction, so that a shortcut could be assigned to it possibly.</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>