<table><tr><td style="">kossebau created this revision.<br />kossebau added reviewers: Kate, KDevelop.<br />Herald added projects: Kate, Frameworks.<br />Herald added subscribers: kde-frameworks-devel, kwrite-devel.<br />kossebau 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/D27533">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Symbols used for marks can be used in different sizes, e.g. depending of the<br />
line height, to which the icon border adapts, or for the context menu on<br />
actions to toggle those marks. Being limited to set a single pixmap as<br />
symbol for a mark results can result in badly scaled symbols being<br />
displayed.<br />
Switching to QIcon as dynamic pixmap provider for markers improves this.</p>

<p>For backward compatibility QIcon & QPixmaps are converted into each other in<br />
case APIs are used mixed.</p>

<p>Currently this is WIP as KDevelop as user of the mark interfaces partially<br />
relies on providing a single pixmap only, and while taking the pixmaps from<br />
QIcons does some QIcon::Mode-based rendering (for breakpoints, marking<br />
disabled, reached, pending or normal ones) or color-tinting (marking line<br />
removal/addition in patch display with positive & negative colors) before<br />
passing the pixmaps on. I am not yet sure whether to<br />
a) turning all those processings into explisit separate dedicatd icons</p>

<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);">(my favourite)</pre></div>

<p>b) adding some additional rendering flags to marks<br />
is the best approach.</p>

<p>If going for a), this patch would be final as is.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>BRANCH</strong><div><div>addmarkinterfacev2</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27533">https://phabricator.kde.org/D27533</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/document/katedocument.cpp<br />
src/document/katedocument.h<br />
src/include/ktexteditor/markinterface.h<br />
src/search/katesearchbar.cpp<br />
src/view/kateviewhelpers.cpp</div></div></div><br /><div><strong>To: </strong>kossebau, Kate, KDevelop<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, rrosch, LeGast00n, cblack, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>