<table><tr><td style="">nibags 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/D18475">View Revision</a></tr></table><br /><div><div><p>Hi!!<br />
Please, also add a test file in: "autotests/input/". To generate the files in "folding", "html" and "reference" use <tt style="background: #ebebeb; font-size: 13px;">make test</tt> and <tt style="background: #ebebeb; font-size: 13px;">autotests/update-reference-data.sh</tt>.</p>

<p>I am not a reviewer, but I want to add some recommendations in relation to the itemData's:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">On lines 467, 476 and 485 you use harded colors for the fonts. Ideally, don't use them, but use the default styles, since a better visualization is achieved in the different schemes: <a href="https://kate-editor.org/2014/03/07/kate-part-kf5-new-default-styles-for-better-color-schemes/" class="remarkup-link" target="_blank" rel="noreferrer">https://kate-editor.org/2014/03/07/kate-part-kf5-new-default-styles-for-better-color-schemes/</a></li>
<li class="remarkup-list-item">On lines 469 & 471 you use background color, and on lines 466-467, 473-474, 479-481 and 490 you use background color, but not font color. Try not to use background color if it's not necessary,  to get a better visualization in the different schemes. However, if you need background color, it also specifies the font color, to avoid readability problems in some color schemes (for example, the background color <tt style="background: #ebebeb; font-size: 13px;">&monospaced_bg;</tt> with <tt style="background: #ebebeb; font-size: 13px;">dsNormal</tt> is not readable in dark schemes).</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R216 Syntax Highlighting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18475">https://phabricator.kde.org/D18475</a></div></div><br /><div><strong>To: </strong>andreasgr, Framework: Syntax Highlighting<br /><strong>Cc: </strong>nibags, kwrite-devel, kde-frameworks-devel, Framework: Syntax Highlighting, bmortimer, hase, michaelh, genethomas, ngraham, bruns, demsking, cullmann, vkrause, sars, dhaumann<br /></div>