Skip to content

Commit f969c05

Browse files
authored
remove now irrelevant jsx v3 stuff from editor tooling (#7158)
1 parent 4a748bb commit f969c05

File tree

3 files changed

+10
-636
lines changed

3 files changed

+10
-636
lines changed

analysis/src/CompletionBackEnd.ml

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1816,9 +1816,6 @@ let rec processCompletable ~debug ~full ~scope ~env ~pos ~forHover completable =
18161816
let keyLabels =
18171817
if Utils.startsWith "key" prefix then [mkLabel ("key", "string")] else []
18181818
in
1819-
(* We always try to look up completion from the actual domProps type first.
1820-
This works in JSXv4. For JSXv3, we have a backup hardcoded list of dom
1821-
labels we can use for completion. *)
18221819
let pathToElementProps = TypeUtils.pathToElementProps package in
18231820
if Debug.verbose () then
18241821
Printf.printf
@@ -1852,12 +1849,7 @@ let rec processCompletable ~debug ~full ~scope ~env ~pos ~forHover completable =
18521849
Printf.printf
18531850
"[completing-lowercase-jsx] could not find element props to complete \
18541851
from.\n";
1855-
(CompletionJsx.domLabels
1856-
|> List.filter (fun (name, _t) ->
1857-
Utils.startsWith name prefix
1858-
&& (forHover || not (List.mem name identsSeen)))
1859-
|> List.map mkLabel)
1860-
@ keyLabels)
1852+
keyLabels)
18611853
| Cjsx (componentPath, prefix, identsSeen) ->
18621854
let labels =
18631855
CompletionJsx.getJsxLabels ~componentPath ~findTypeOfValue ~package

0 commit comments

Comments
 (0)