- 
                Notifications
    
You must be signed in to change notification settings  - Fork 471
 
Closed
Description
<CompletionSupport.TestComponent on= someOtherProp=trueThis is parsed as:
[
  structure_item 
    Pstr_eval
    expression 
      Pexp_apply
      expression 
        Pexp_ident "React.createElement" 
      [
        <arg>
        Nolabel
          expression 
            Pexp_ident "CompletionSupport.TestComponent.make" 
        <arg>
        Nolabel
          expression 
            Pexp_record
            [
              "on" 
                expression 
                  attribute  "res.namedArgLoc"
                    []
                  Pexp_ident "someOtherProp" 
            ]
            None
      ]
  structure_item 
    Pstr_eval
    expression 
      Pexp_construct "true" 
      None
]
...which effectively means:
<CompletionSupport.TestComponent on=someOtherProp
trueBut, preferably it should be parsed as:
<CompletionSupport.TestComponent on=%rescript.exprHole someOtherProp=trueThe current state prevents us from doing proper completions in this case, which is quite common, so it's a bit annoying.