<table><tr><td style="">ngraham added a reviewer: VDG.<br />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>Thanks <a href="https://phabricator.kde.org/p/simgunz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@simgunz</a>, this is really excellent work. Adding <span class="phabricator-remarkup-mention-unknown">@VDG</span> for more UI review comments.</p>

<p>For my own UI review, I must continue to push for putting the annotation tools on a second toolbar that appears below the main one when needed. Adding them to the main toolbar is only feasible when the window is really really wide; otherwise most or all of the buttons get elided. Once they're in their own toolbar, there should be enough horizontal space to give most or all of the buttons text on the side, which should resolve the confusion regarding what they are and which ones are actions vs settings for other tools.</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>ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen<br /></div>