<table><tr><td style="">simgunz edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ej6eo4whg7lbvja/">(Show Details)</a>
</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>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Fix design of text annotation tools<br />
<br />
Fix layout of freehand tool<br />
<br />
Fix layout of highlighter<br />
<br />
Fix layout of geometrical shape<br />
<br />
Fix layout of straight line and polygon<br />
<br />
# Conflicts:<br />
#       ui/annotationwidgets.cpp<br />
<br />
Position spacer at the bottom of the grid layout<br />
<br />
Simplify code<br />
<br />
# Conflicts:<br />
#       ui/annotationwidgets.cpp<br />
<br />
Fix layout of stamp<br />
<br />
Fix line endings<br />
<br />
Fix font capitalization<br />
<br />
Remove unuseful return<br />
<br />
Add icon label<br />
<br />
Fix indentation<br />
<br />
Fix layout of caret annotation<br />
<br />
Remove unused code<br />
<br />
Fix layout of file attachment<br />
<br />
Rename layout > gridLayout for clarity<br />
<br />
Make row variable constant<br />
<br />
Set buddy<br />
<br />
Mark variable unused<br />
<br />
Fix layout of extraWidget of FileAttachment<br />
<br />
Fix layout of General Info<br />
<br />
Use form layout instead of grid layout<br />
<br />
Set label alignment and field growth policy<br />
<br />
Resort code of line annotation<br />
<br />
Fix wrong widget set as parent<br />
<br />
Move line termination styles before leaders style<br />
<br />
Align combobox to top<br />
<br />
Fix label buddy not working of PixmapPreviewSelector<br />
<br />
Fix keyboard shortcut of Author conflicts with Add<br />
<br />
Same order of Size and Fill color for GeomShape and Polygon<br />
<br />
Make annotation type the first element<br />
<br />
Add color button and opacity spin manually in each annotation<br />
<br />
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</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Created by mistake</span>. <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">It also removes the need of the custom methods<br />
hasColorButton and hasOpacityBox<br />
<br />
Add vertical space to separate groups of settings<br />
<br />
Add opacity spin to file attachment annotation<br />
<br />
Rename "Size" to "Width"<br />
<br />
Set better accelerators<br />
<br />
Change string that inform the user about how Identity info are used<br />
<br />
Remove only-for-pdf tooltips<br />
<br />
Set accelerator explicitly<br />
<br />
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</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">See D10859.</span></div></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/D21730">https://phabricator.kde.org/D21730</a></div></div><br /><div><strong>To: </strong>simgunz<br /><strong>Cc: </strong>shubham, ngraham, okular-devel, joaonetto, tfella, darcyshen, aacid<br /></div>