<table><tr><td style="">simgunz updated this revision to Diff 67517.<br />simgunz 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><ul class="remarkup-list">
<li class="remarkup-list-item">Rename favorite > quick in settings</li>
<li class="remarkup-list-item">Refactor</li>
<li class="remarkup-list-item">Clean pageviewutils</li>
<li class="remarkup-list-item">Increase part.rc version</li>
<li class="remarkup-list-item">Improve tools defaults</li>
<li class="remarkup-list-item">Clean code</li>
<li class="remarkup-list-item">Remove unused method</li>
<li class="remarkup-list-item">Rename methods</li>
<li class="remarkup-list-item">Rename, move and comment methods</li>
<li class="remarkup-list-item">Fix quickTools</li>
<li class="remarkup-list-item">Move methods</li>
<li class="remarkup-list-item">Move / rename methods</li>
<li class="remarkup-list-item">Make methods private</li>
<li class="remarkup-list-item">Update description</li>
<li class="remarkup-list-item">Move methods to original position</li>
<li class="remarkup-list-item">Improve checks</li>
<li class="remarkup-list-item">Minor style change</li>
<li class="remarkup-list-item">Rename methods favorite > quick</li>
<li class="remarkup-list-item">Reorder methods</li>
<li class="remarkup-list-item">Move methods to private class</li>
<li class="remarkup-list-item">Correctly disable all actions when annotations not allowed</li>
<li class="remarkup-list-item">Rename variables</li>
<li class="remarkup-list-item">Indent</li>
<li class="remarkup-list-item">Clean parseTool</li>
<li class="remarkup-list-item">Code style fixes</li>
<li class="remarkup-list-item">Clean constructor</li>
<li class="remarkup-list-item">Remove unuseful signal</li>
<li class="remarkup-list-item">Simplify code to set picker color</li>
<li class="remarkup-list-item">Clean tooltips method</li>
<li class="remarkup-list-item">Clean populate quick annotation</li>
<li class="remarkup-list-item">Clean icons generator method</li>
<li class="remarkup-list-item">Reorder methods</li>
<li class="remarkup-list-item">Rename method</li>
<li class="remarkup-list-item">Simplify deselecting tool code</li>
<li class="remarkup-list-item">Put similar methods close together</li>
<li class="remarkup-list-item">Add method description</li>
<li class="remarkup-list-item">Remove unuseful connection</li>
<li class="remarkup-list-item">Remove comment</li>
<li class="remarkup-list-item">Add comments regarding action group workaround</li>
<li class="remarkup-list-item">Refactor stamp tool code</li>
<li class="remarkup-list-item">Comments</li>
<li class="remarkup-list-item">Remove unused code</li>
<li class="remarkup-list-item">Drop unuseful slots</li>
<li class="remarkup-list-item">Simplify updateConfigActions method</li>
<li class="remarkup-list-item">Use new edit-line-width icon</li>
<li class="remarkup-list-item">Refactor method to set color picker icon</li>
<li class="remarkup-list-item">Fix stamp tool not properly selected</li>
<li class="remarkup-list-item">Refactor parseTool</li>
<li class="remarkup-list-item">Refactor updateConfigActions: remove none, no annotation tooltips</li>
<li class="remarkup-list-item">Use isValid instead of setting Qt::transparent</li>
<li class="remarkup-list-item">Spaces</li>
<li class="remarkup-list-item">Move instruction at the end</li>
<li class="remarkup-list-item">Refactor insertion of custom action (width and opacity)</li>
<li class="remarkup-list-item">Move code to select stamp action in a method</li>
<li class="remarkup-list-item">Style</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D15580?vs=67036&id=67517">https://phabricator.kde.org/D15580?vs=67036&id=67517</a></div></div><br /><div><strong>BRANCH</strong><div><div>annotation-toolbar-refactor</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>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
conf/dlgannotations.cpp<br />
conf/dlgannotationsbase.ui<br />
conf/editannottooldialog.cpp<br />
conf/editannottooldialog.h<br />
conf/okular.kcfg<br />
okular.upd<br />
part.cpp<br />
part.rc<br />
shell/shell.cpp<br />
shell/shell.h<br />
shell/shell.rc<br />
ui/annotationactionhandler.cpp<br />
ui/annotationactionhandler.h<br />
ui/annotationwidgets.cpp<br />
ui/annotationwidgets.h<br />
ui/data/CMakeLists.txt<br />
ui/data/tools.xml<br />
ui/data/toolsQuick.xml<br />
ui/pageview.cpp<br />
ui/pageview.h<br />
ui/pageviewannotator.cpp<br />
ui/pageviewannotator.h<br />
ui/pageviewutils.cpp<br />
ui/pageviewutils.h<br />
ui/side_reviews.cpp<br />
ui/toggleactionmenu.cpp<br />
ui/toggleactionmenu.h<br />
ui/toolaction.cpp<br />
ui/toolaction.h</div></div></div><br /><div><strong>To: </strong>simgunz, Okular, VDG<br /><strong>Cc: </strong>simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, andisa, siddharthmanthan, maguirre, fbampaloukas, joaonetto, kezik, tfella, darcyshen<br /></div>