martonmiklos added a comment.

  In https://phabricator.kde.org/D7367#136687, @cullmann wrote:
  
  > Could you add a minimal example as unit test?
  
  
  Sure, I have just seen that these xmls could be autotested.
  
  > And could we have as license something like "MIT"?
  
  Sure, I have no objections about it.
  
  > For the coloring: it is preferred to use some of our default styles and no 
hard coded colors to make color schemes possible.
  
  
  
  In https://phabricator.kde.org/D7367#136817, @dhaumann wrote:
  
  > Please change the kateversion, and as Christoph mentioned please remove 
hard-coded colors. You can find a list of all valid default styles here: 
https://kate-editor.org/2014/03/07/kate-part-kf5-new-default-styles-for-better-color-schemes/
  
  
  Thanks for the tips, I will change the license, colors and add autotest.

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D7367

To: martonmiklos, #framework_syntax_hightlighting
Cc: dhaumann, cullmann, #frameworks, vkrause

Reply via email to