jpoelen added a comment.

  Sorry to answer so late, I had trouble logging in.
  
  I think I misinterpreted the use of `includedDefinitions()`. For me, this 
function lists the languages ​included by the rules and which influence the 
colors. While the inclusion of a keyword list has no influence here. In the 
case of SASS, there is no IncludeRules to CSS because small syntactic details 
do not allow it, it would not be logical to have CSS in the list of 
`includedDefinitions()`.
  
  Basically, I added `includedKeywordDefinitions()` because of a Format test 
that did not work. But it's true that putting it in the public interface is not 
a good idea.

REPOSITORY
  R216 Syntax Highlighting

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

To: jpoelen, #framework_syntax_highlighting, cullmann, dhaumann
Cc: kwrite-devel, vkrause, kde-frameworks-devel, michaelh, kevinapavew, 
ngraham, bruns, demsking, cullmann, sars, dhaumann

Reply via email to