Add operators subscopes

[MaximSokolov]

Closes #188
Supersedes and closes #222

It's not correct for some cases: Has been reverted

// ':' isn't `keyword.operator.conditional.ternary`
var obj = { a: "b"}

// e.g. '+' operator can be used for concatenation
// it's `keyword.operator.arithmetic`
"str1" + "str2"
var1 + var2

[winstliu]

Can I have your reasoning behind this change?

[MaximSokolov]

It makes possible to change highlighting for each of these operators.

[MaximSokolov]

Changed:

• Revert ?, : to keyword.operator
• keyword.operator.assignment for : (when it is assignment of course)

augmented sounds really weird to me :/
Anyone have suggestions for a different name?

Python, Ruby use assignment.augmented I guess it means assignment augmented by another operator.

[MaximSokolov]

Should I revert 'keyword.operator.arithmetic' -> 'keyword.operator' for *, /, -, %, + operators?

[Victorystick]

@MaximSokolov That sounds like a good idea. We can't be sure that a + is used for arithmetic or concatenation. :)

[winstliu]

@MaximSokolov I'd suggest to change it as well.

[MaximSokolov]

Is keyword.operator.assignment.compound ok for %=, +=, -=, *= ... ?
(instead of ...assignment.augmented)

[winstliu]

Sounds better to me 😆.
/cc @atom/feedback

[mehcode]

As a (self proclaimed) language designer, compound is more correct (as you are joining two distinct operators into a new operator, now composed of two parts). People started calling it augmented with scripting languages (C, Java, etc. always called it compound).

[winstliu]

: doesn't need to be escaped

[winstliu]

-- doesn't need to be escaped

[winstliu]

Final nit: The - here doesn't need to be escaped either.

[winstliu]

Woo!

[smaili]

👍