<table><tr><td style="">simgunz created this revision.<br />Herald added a project: Okular.<br />Herald added a subscriber: okular-devel.<br />simgunz requested review of this revision.
</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/D21730">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Fix design of text annotation tools</p>

<p>Fix layout of freehand tool</p>

<p>Fix layout of highlighter</p>

<p>Fix layout of geometrical shape</p>

<p>Fix layout of straight line and polygon</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Conflicts:</li>
<li class="remarkup-list-item">ui/annotationwidgets.cpp</li>
</ol>

<p>Position spacer at the bottom of the grid layout</p>

<p>Simplify code</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Conflicts:</li>
<li class="remarkup-list-item">ui/annotationwidgets.cpp</li>
</ol>

<p>Fix layout of stamp</p>

<p>Fix line endings</p>

<p>Fix font capitalization</p>

<p>Remove unuseful return</p>

<p>Add icon label</p>

<p>Fix indentation</p>

<p>Fix layout of caret annotation</p>

<p>Remove unused code</p>

<p>Fix layout of file attachment</p>

<p>Rename layout > gridLayout for clarity</p>

<p>Make row variable constant</p>

<p>Set buddy</p>

<p>Mark variable unused</p>

<p>Fix layout of extraWidget of FileAttachment</p>

<p>Fix layout of General Info</p>

<p>Use form layout instead of grid layout</p>

<p>Set label alignment and field growth policy</p>

<p>Resort code of line annotation</p>

<p>Fix wrong widget set as parent</p>

<p>Move line termination styles before leaders style</p>

<p>Align combobox to top</p>

<p>Fix label buddy not working of PixmapPreviewSelector</p>

<p>Fix keyboard shortcut of Author conflicts with Add</p>

<p>Same order of Size and Fill color for GeomShape and Polygon</p>

<p>Make annotation type the first element</p>

<p>Add color button and opacity spin manually in each annotation</p>

<p>This allows to add each row in order without the need to insert at position 0<br />
the row with the "Type" for Highlighter and GeomAnnotation. Moreover it is not<br />
possible to insert spacer items at a given row with addItem, so we cannot put a<br />
space after the "Type" row. It also removes the need of the custom methods<br />
hasColorButton and hasOpacityBox</p>

<p>Add vertical space to separate groups of settings</p>

<p>Add opacity spin to file attachment annotation</p>

<p>Rename "Size" to "Width"</p>

<p>Set better accelerators</p>

<p>Change string that inform the user about how Identity info are used</p>

<p>Remove only-for-pdf tooltips</p>

<p>Set accelerator explicitly</p>

<p>Set of letter 't' because 'a' and 'u' are used for Add and Move Up in widgetconfigurationtoolsbase.cpp which will appear in the same UI</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-annot-config-dialog</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21730">https://phabricator.kde.org/D21730</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>conf/dlgannotationsbase.ui<br />
ui/annotationpropertiesdialog.cpp<br />
ui/annotationwidgets.cpp<br />
ui/annotationwidgets.h</div></div></div><br /><div><strong>To: </strong>simgunz<br /><strong>Cc: </strong>okular-devel, joaonetto, tfella, ngraham, darcyshen, aacid<br /></div>