<table><tr><td style="">croick 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/D8709">View Revision</a></tr></table><br /><div><div><p>I like it so far. The additional information is certainly helpful when trying to understand the history of some piece of code.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The behaviour of the ToolTip is better now. However, if the commit message is too short, it's sometimes not possible to reach "Select All" in the context menu of the ToolTip. But that's rather an issue with the <tt style="background: #ebebeb; font-size: 13px;">ActiveToolTip</tt> vanishing immediatly once it's left.</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">If I try to "Show Diff..." of any annotation, KDevelop will crash:</li>
</ul>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">#6  0x00007f1e74ff3d37 in KTextEditor::ViewPrivate::uniformAnnotationItemSizes() const (this=0x562425e5c750) at /home/christoph/Software/kde/frameworks/ktexteditor/src/view/kateviewhelpers.h:299
#7  0x00007f1e75016b63 in KateIconBorder::calcAnnotationBorderWidth() (this=this@entry=0x56242e3eb9f0) at /home/christoph/Software/kde/frameworks/ktexteditor/src/view/kateviewhelpers.cpp:2722
#8  0x00007f1e75016ca0 in KateIconBorder::updateFont() (this=0x56242e3eb9f0) at /home/christoph/Software/kde/frameworks/ktexteditor/src/view/kateviewhelpers.cpp:1631
#9  0x00007f1e7501b092 in KateIconBorder::KateIconBorder(KateViewInternal*, QWidget*) (this=0x56242e3eb9f0, internalView=<optimized out>, parent=<optimized out>) at /home/christoph/Software/kde/frameworks/ktexteditor/src/view/kateviewhelpers.cpp:1458
#10 0x00007f1e75005f00 in KateViewInternal::KateViewInternal(KTextEditor::ViewPrivate*) (this=0x56242a732190, view=<optimized out>) at /home/christoph/Software/kde/frameworks/ktexteditor/src/view/kateviewinternal.cpp:233
#11 0x00007f1e74ffe904 in KTextEditor::ViewPrivate::ViewPrivate(KTextEditor::DocumentPrivate*, QWidget*, KTextEditor::MainWindow*) (this=0x562425e5c750, doc=0x562424247cc0, parent=<optimized out>, mainWindow=0x562423ba63f0) at /home/christoph/Software/kde/frameworks/ktexteditor/src/document/katedocument.h:1049
#12 0x00007f1e74f77f58 in KTextEditor::DocumentPrivate::createView(QWidget*, KTextEditor::MainWindow*) (this=this@entry=0x562424247cc0, parent=parent@entry=0x56242dc32310, mainWindow=0x562423ba63f0) at /home/christoph/Software/kde/frameworks/ktexteditor/src/document/katedocument.cpp:417
#13 0x00007f1e764f5207 in KDevelop::TextDocument::createViewWidget(QWidget*) (this=0x5624259b14c0, parent=0x56242dc32310) at /usr/include/c++/8.2.1/bits/atomic_base.h:390
#14 0x00007f1e764f4225 in KDevelop::TextView::createWidget(QWidget*) (this=0x56242635b410, parent=0x56242dc32310) at /home/christoph/Software/kde/extragear/kdevelop/kdevelop/kdevplatform/shell/textdocument.cpp:565</pre></div></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D8709#inline-83546">View Inline</a><span style="color: #4b4d51; font-weight: bold;">vcsannotationitemdelegate.cpp:388</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #74777d">// Ideally the user could configure the width of the annotations, best interactively.</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #74777d">// Until this is possible, the sizehint is: roughly 40 chars, but maximal 25 % of the view</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #74777d">// See eventFilter for making sure we adapt the max 25 % to a changed width.</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">On a small screen, that's taking a little too much space in my opinion. I preferred 20%.</p>

<p style="padding: 0; margin: 8px;">What would be necessary to make it resizable?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8709">https://phabricator.kde.org/D8709</a></div></div><br /><div><strong>To: </strong>kossebau, kfunk<br /><strong>Cc: </strong>croick, kfunk, brauch, apol, kdevelop-devel<br /></div>