D18951: HTML: highlight JSX, TypeScript & MustacheJS code in the <script> tag
Christoph Cullmann
noreply at phabricator.kde.org
Tue Feb 12 19:29:13 GMT 2019
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
I think this is a useful addition, the license is ok, there is a test case and no old tests regress.
I am not sure how good the coverage for the generated highlightings is, but at least no obvious issues.
Thanks for the contribution.
Btw., if you have more insight about the quality of the tests for the generated hl files and can improve that, a extra patch with more autotests is appreciated.
But maybe the coverage is already good ;=)
+1 for no hard-coded colors ;=)
REPOSITORY
R216 Syntax Highlighting
BRANCH
html-script
REVISION DETAIL
https://phabricator.kde.org/D18951
To: nibags, #framework_syntax_highlighting, dhaumann, cullmann
Cc: kwrite-devel, kde-frameworks-devel, gennad, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190212/275198d8/attachment.html>
More information about the Kde-frameworks-devel
mailing list