🐛 Don't scope decimals as illegal

Author: MaximSokolov

grammars/javascript.cson

@@ -1211,15 +1211,13 @@
     'patterns': [
       {
         'comment': '`prop` in `obj.prop`, `func().prop`'
-        'match': '(\\.)\\s*(?:(\\b\\d\\w*)|([A-Z][A-Z0-9_$]*\\b\\$*)|([\\w$]*))'
+        'match': '(\\.)\\s*(?:([A-Z][A-Z0-9_$]*\\b\\$*)|(\\$*[a-zA-Z_$][\\w$]*))'
         'captures':
           '1':
             'name': 'meta.delimiter.property.period.js'
           '2':
-            'name': 'invalid.illegal.js'
-          '3':
             'name': 'constant.other.property.js'
-          '4':
+          '3':
             'name': 'variable.other.property.js'
       }
     ]

spec/javascript-spec.coffee

@@ -1051,10 +1051,6 @@ describe "Javascript grammar", ->
         {tokens} = grammar.tokenizeLine('123a.prop')
         expect(tokens[0]).toEqual value: '123a', scopes: ['source.js', 'invalid.illegal.js']
 
-      it "doesn't confuse illegal objects with numbers", ->
-        {tokens} = grammar.tokenizeLine('123.')
-        expect(tokens[0]).toEqual value: '123', scopes: ['source.js', 'constant.numeric.js']
-
   describe "function calls", ->
     it "tokenizes function calls", ->
       {tokens} = grammar.tokenizeLine('functionCall()')
@@ -1220,11 +1216,6 @@ describe "Javascript grammar", ->
       expect(tokens[3]).toEqual value: '.', scopes: ['source.js', 'meta.delimiter.property.period.js']
       expect(tokens[4]).toEqual value: 'b', scopes: ['source.js', 'variable.other.property.js']
 
-      {tokens} = grammar.tokenizeLine('a.123illegal')
-      expect(tokens[0]).toEqual value: 'a', scopes: ['source.js']
-      expect(tokens[1]).toEqual value: '.', scopes: ['source.js', 'meta.delimiter.property.period.js']
-      expect(tokens[2]).toEqual value: '123illegal', scopes: ['source.js', 'invalid.illegal.js']
-
     it "tokenizes constant properties", ->
       {tokens} = grammar.tokenizeLine('obj.MY_CONSTANT')
       expect(tokens[0]).toEqual value: 'obj', scopes: ['source.js', 'variable.other.object.js']