<table><tr><td style="">loh.tar updated this revision to Diff 49636.<br />loh.tar 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/D18164">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Fix typo of member vars</li>
<li class="remarkup-list-item">Add missing setView(m_view)</li>
<li class="remarkup-list-item">Remove FIXME hint about timer, however would a comment for me nice</li>
<li class="remarkup-list-item">Use QString()</li>
</ul>

<p>Things which could be improved or not</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The up/down buttons are a little bit small, have tried to give them the hight (and width) of the hight of the other buttons, but without    success. Ideas?</li>
<li class="remarkup-list-item">There is no tooltip on the spinbox and Goto button</li>
<li class="remarkup-list-item">The spinbox will not adjust its size and value boundaries when you paste text while its shown. Try:<ul class="remarkup-list">
<li class="remarkup-list-item">New file->show bar</li>
<li class="remarkup-list-item">Paste 10+ lines</li>
<li class="remarkup-list-item">Spinbox keep size of one digit and can't be set to an other value as 1</li>
</ul></li>
</ul></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D18164?vs=49243&id=49636">https://phabricator.kde.org/D18164?vs=49243&id=49636</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18164">https://phabricator.kde.org/D18164</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/dialogs/katedialogs.cpp<br />
src/dialogs/katedialogs.h</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann<br /><strong>Cc: </strong>dhaumann, cullmann, anthonyfieroni, kwrite-devel, kde-frameworks-devel, KTextEditor, hase, michaelh, ngraham, bruns, demsking, sars<br /></div>