<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>getting there!</p>

<p>UI review of the latest version:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Having the annotation tools on the main toolbar makes it overflow for all but the largest window sizes. How about putting them in a secondary toolbar below the main one that's hidden until the user shows it by clicking on a "Show annotation tools" item on the main toolbar and/or menubar?</li>
<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 ">Line width</span></span></span> dropdown menu button should make the whole button clickable to show the drop-down rather than only the space under the arrow on the right</li>
<li class="remarkup-list-item">The entries in the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Line width</span></span></span> dropdown should visually reflect the width of the line. Could these graphics be generated programmatically rather than using icons?</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 ">Squiggle</span></span></span> and <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Arrow</span></span></span> need new icons; please file a bug on Breeze | Icons and request them</li>
<li class="remarkup-list-item">The <tt style="background: #ebebeb; font-size: 13px;">draw-polygon</tt> icon might make it seem like that tool can only draw pentagons, consider using <tt style="background: #ebebeb; font-size: 13px;">draw_polyline</tt> or <tt style="background: #ebebeb; font-size: 13px;">draw-polygon-star</tt> instead</li>
<li class="remarkup-list-item">The Color button should actually show the current color rather than a generic icon</li>
<li class="remarkup-list-item">The Inner color button needs something to show that it exists; right now it just looks like whitespace in the toolbar. It should show its color like the other button, and for "no color", maybe an empty transparent square?</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 ">Straight Line</span></span></span> is mis-named, it's for drawing anything <em>but</em> a straight line! :) Should be something like "Freehand line"</li>
<li class="remarkup-list-item">I can't figure out what <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Pin Annotation</span></span></span> actually does</li>
<li class="remarkup-list-item">It's not clear to me how to select existing annotations once an annotation tool has been activated; consider maybe adding a "select annotations" tool or mode under the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Selection</span></span></span> dropdown menu.</li>
</ul></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<br /><strong>Cc: </strong>knambiar, ngraham, tobiasdeiminger, okular-devel, joaonetto, tfella, darcyshen, aacid<br /></div>