Move LSL keyword default to lsl_definitions.yaml - #123
Conversation
default from constants to controls
default from constants to controlsdefault from constants to controls
|
It's really only under there so the color of |
|
it's not breaking anything, in that:
Why somebody wanted it to have a different color than The motivation for this PR is indeed weak: removing a one-line exception from the textmate generator, and 7 lines of exception elsewhere While I don't have a strong opinion about the highlighting color (bikeshed topic), it would be nice to move |
|
Personally i agree it should be in flow control. But i do think it needs a special case to be kept in constants for now for I'm pretty certain that loads of less familiar scripters would think its an error or just miss it being blue 😢 |
|
moved it back to |
default from constants to controlsdefault to the lsl_definitions.yaml
default to the lsl_definitions.yamldefault to lsl_definitions.yaml
defaultwas misplaced in the keywords file as a constant, rather than a control, and the generators had some special handling for it. Move it to the logical place, and remove the exceptionsThis also removes exceptional handling for it in the unmerged textmate generator: #120 (review)
This changes it's in-viewer highlighting color from blue to purple