<div dir="ltr">Sorry for the long silence. I have been busy at work and then on vacation.<div><br></div><div>Personally I do not like the idea of the vertical toolbar specifically for Okular for the following reasons:<div>- Okular differently from Gwenview already has a tab bar, which means that further splitting the reviews tab into two pieces add another nesting level. </div><div>- Splitting the Review tab horizontally with the toolbar on top and the reviews below seems quite cluttered to me. Also that review tab could be improved (see T8553) and may require all the vertical space (e.g. show the full text of the notes, allow to reply, etc. See Foxit for example).</div><div>-  It is not possible to build the toolbar with KXMLGUI as it is now, but needs to be built programmatically (I think). The Gwenview toolbar is not a standard toolbar, indeed it cannot be edited or moved.</div><div><br></div><div>The reasons for which I prefer to make the annotation toolbar a standard toolbar displaying only the icons are:</div><div>- Many other programs have similar icon-only toolbars (e.g. libreoffice, and also other PDF viewers e.g. MacOS viewer and Adobe reader). For the annotation toolbar I would conform with the other PDF viewers.</div><div>- The icon buttons have tooltips and likely the users will quickly learn them</div></div><div>- The toolbar is standard and can be customized by the user.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 20, 2019 at 6:26 PM Nathaniel Graham <<a href="mailto:noreply@phabricator.kde.org" target="_blank">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><table><tbody><tr><td>ngraham added a comment.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0px 8px 8px;float:right;color:rgb(70,76,92);font-weight:bold;border-radius:3px;background-color:rgb(247,247,249);background-image:linear-gradient(rgb(255,255,255),rgb(241,240,241));display:inline-block;border:1px solid rgba(71,87,120,0.2)" href="https://phabricator.kde.org/D15580" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><blockquote style="border-left:3px solid rgb(140,152,184);color:rgb(107,116,140);font-style:italic;margin:4px 0px 12px;padding:8px 12px;background-color:rgb(248,249,252)">
<div style="font-style:normal;padding-bottom:4px">In <a href="https://phabricator.kde.org/D15580#515250" style="background-color:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">D15580#515250</a>, <a href="https://phabricator.kde.org/p/davidhurka/" style="color:rgb(25,85,141);background-color:rgb(241,247,255);border:1px solid transparent;border-radius:3px;font-weight:bold;padding:0px 4px" target="_blank">@davidhurka</a> wrote:</div>
<div style="margin:0px;padding:0px;border:0px;color:rgb(107,116,140)"><blockquote style="border-left:3px solid rgb(140,152,184);color:rgb(107,116,140);font-style:italic;margin:4px 0px 12px;padding:8px 12px;background-color:rgb(248,249,252)">
<div style="font-style:normal;padding-bottom:4px">In <a href="https://phabricator.kde.org/D15580#514707" style="background-color:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">D15580#514707</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="color:rgb(25,85,141);background-color:rgb(241,247,255);border:1px solid transparent;border-radius:3px;font-weight:bold;padding:0px 4px" target="_blank">@ngraham</a> wrote:</div>
<div style="margin:0px;padding:0px;border:0px;color:rgb(107,116,140)"><blockquote style="border-left:3px solid rgb(140,152,184);color:rgb(107,116,140);font-style:italic;margin:4px 0px 12px;padding:8px 12px;background-color:rgb(248,249,252)">
<div style="font-style:normal;padding-bottom:4px">In <a href="https://phabricator.kde.org/D15580#513826" style="background-color:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">D15580#513826</a>, <a href="https://phabricator.kde.org/p/simgunz/" style="color:rgb(25,85,141);background-color:rgb(241,247,255);border:1px solid transparent;border-radius:3px;font-weight:bold;padding:0px 4px" target="_blank">@simgunz</a> wrote:</div>
<div style="margin:0px;padding:0px;border:0px;color:rgb(107,116,140)"><ul>
<li>How would you fit the annotation actions in the <span><span><span>Reviews</span></span></span> tab?</li>
<li>Would you create a sub-tab in it (as in Gwenview where the tabs are at the bottom)? -</li>
<li>Can you provide a minimal mockup of this?</li>
</ul></div>
</blockquote>

<p>Having it tabbed like Gwenview was what I was envisioning, yeah. Basically copy the UX of Gwenview's sidebar, but inside Okular's <span><span><span>Reviews</span></span></span> tab.</p></div>
</blockquote>

<p>I’m not sure whether I understand you. This is a screenshot of the sidebar in Gwenview “<span><span><span>Operations</span></span></span>”. Additionally to the sidebar tab <span><span><span>Reviews</span></span></span>, you want a tab “<span><span><span>Add Annotations</span></span></span>”, looking like this?<br>
 <a href="https://phabricator.kde.org/F7264576" style="background-color:rgb(231,231,231);border-color:rgb(231,231,231);border-radius:3px;padding:0px 4px;font-weight:bold;color:black;text-decoration:none" target="_blank">F7264576: image.png</a><br>
 Or you want to add a tab bar to the bottom of the <span><span><span>Reviews</span></span></span> tab, containing <span><span><span>Annotations</span></span></span> and <span><span><span>Add Annotations</span></span></span>?<br>
 I would simply put the annotation toolbar on top of the <span><span><span>Reviews</span></span></span> tab, on top of the search box. Probably it would cover multiple lines, if that is possible with Qt.</p></div>
</blockquote>

<p>Yeah, that might make more sense that having a tabbed view inside the review tab. But yes, I was envisioning a vertical toolbar like the above screenshot.</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" target="_blank">https://phabricator.kde.org/D15580</a></div></div><br><div><strong>To: </strong>simgunz, Okular, VDG<br><strong>Cc: </strong>ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen<br></div></blockquote></div>