@FMuro sorry for the delay simply missed your comment above, just ping if you 
don't get responses in a few weeks and you are still interested, there is such 
a thing as being _too_ patient waiting 6 months :grin: 

What I meant (and I presume @b4n meant) by "does the wrong thing" is that it 
mixes semantics by using "type" for "identifiers". The "proper" fix is for the 
Scintilla lexer to style keys as something else than "identifier", and the 
notebook++ lexer offered on the Scintilla issue from #1853 seems to do that, 
adding a new "key" semantic.  But AFAICT that new lexer hasn't happened yet.

On the other hand YAML does not have the concept of "type" so if it suits YAML 
users I guess it can't hurt to use "type" highlighting for "identifiers" which 
means keys IIUC.  @b4n any comment, or will merge in a week or two.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1838#issuecomment-787608950

Reply via email to