From e249fa32abf3085e1077f5d2881eb06fc291ca11 Mon Sep 17 00:00:00 2001
From: nojaf <florian.verdonck@outlook.com>
Date: Mon, 25 Aug 2025 14:33:57 +0200
Subject: [PATCH 1/5] Change placement of closing > in JSX

---
 compiler/syntax/src/res_printer.ml            | 21 ++-----------------
 .../reason/expected/bracedJsx.res.txt         |  3 ++-
 .../data/printer/expr/expected/jsx.res.txt    | 14 +++++++++++--
 tests/syntax_tests/data/printer/expr/jsx.res  |  9 +++++++-
 .../other/expected/signaturePicker.res.txt    |  6 ++++--
 5 files changed, 28 insertions(+), 25 deletions(-)

diff --git a/compiler/syntax/src/res_printer.ml b/compiler/syntax/src/res_printer.ml
index a7c95cf5ee..a345d59655 100644
--- a/compiler/syntax/src/res_printer.ml
+++ b/compiler/syntax/src/res_printer.ml
@@ -4489,19 +4489,6 @@ and print_jsx_container_tag ~state tag_name
     (closing_tag : Parsetree.jsx_closing_container_tag option)
     (pexp_loc : Location.t) cmt_tbl =
   let name = print_jsx_name tag_name.txt in
-  let last_prop_has_comment_after =
-    let rec visit props =
-      match props with
-      | [] -> None
-      | [x] -> Some x
-      | _ :: xs -> visit xs
-    in
-    let last_prop = visit props in
-    match last_prop with
-    | None -> false
-    | Some last_prop ->
-      has_trailing_comments cmt_tbl (ParsetreeViewer.get_jsx_prop_loc last_prop)
-  in
   let opening_greater_than_loc =
     {
       Warnings.loc_start = opening_greater_than;
@@ -4509,7 +4496,7 @@ and print_jsx_container_tag ~state tag_name
       loc_ghost = false;
     }
   in
-  let opening_greater_than_has_leading_comments, opening_greater_than_doc =
+  let _opening_greater_than_has_leading_comments, opening_greater_than_doc =
     let has_leading_comments =
       has_leading_comments cmt_tbl opening_greater_than_loc
     in
@@ -4577,11 +4564,7 @@ and print_jsx_container_tag ~state tag_name
                    (* if the element name has a single comment on the same line, force newline before '>' *)
                    if has_trailing_single_line_comment cmt_tbl tag_loc then
                      Doc.concat [Doc.hard_line; opening_greater_than_doc]
-                   else if
-                     last_prop_has_comment_after
-                     || opening_greater_than_has_leading_comments
-                   then Doc.concat [Doc.soft_line; opening_greater_than_doc]
-                   else opening_greater_than_doc
+                   else Doc.concat [Doc.soft_line; opening_greater_than_doc]
                  in
                  Doc.concat
                    [
diff --git a/tests/syntax_tests/data/conversion/reason/expected/bracedJsx.res.txt b/tests/syntax_tests/data/conversion/reason/expected/bracedJsx.res.txt
index b49e82fece..17d75e3d9a 100644
--- a/tests/syntax_tests/data/conversion/reason/expected/bracedJsx.res.txt
+++ b/tests/syntax_tests/data/conversion/reason/expected/bracedJsx.res.txt
@@ -111,7 +111,8 @@ let make = () => {
     <div
       className=Styles.terminal
       onClick={event => (event->ReactEvent.Mouse.target)["querySelector"]("input")["focus"]()}
-      ref={containerRef->ReactDOMRe.Ref.domRef}>
+      ref={containerRef->ReactDOMRe.Ref.domRef}
+    >
       {state.history
       ->Array.mapWithIndex((index, item) =>
         <div key={j`$index`} className=Styles.line>
diff --git a/tests/syntax_tests/data/printer/expr/expected/jsx.res.txt b/tests/syntax_tests/data/printer/expr/expected/jsx.res.txt
index 2bf6cdf045..5817e35ee5 100644
--- a/tests/syntax_tests/data/printer/expr/expected/jsx.res.txt
+++ b/tests/syntax_tests/data/printer/expr/expected/jsx.res.txt
@@ -116,7 +116,8 @@ let avatarSection =
           onMouseLeave={_ => setHoveringAdmin(false)}
           onClick={_e => {
             stopImpersonating(csrfToken)
-          }}>
+          }}
+        >
           <Avatar user={viewer} size={45} />
         </div>
       : React.nullElement}
@@ -497,7 +498,8 @@ let x =
       className="sdf sdfdsf sdfs sdf asdf dsf"
       onClick={() => {
         ()
-      }}>
+      }}
+    >
       <span
         id="2-1"
         className="sdf sdfdsf sdfs sdf asdf dsf"
@@ -566,3 +568,11 @@ let arrow_with_container_tag = el =>
     el
     {t(")")}
   </div>
+
+<Component
+  prop1
+  //
+  prop2
+>
+  {React.string("yow")}
+</Component>
diff --git a/tests/syntax_tests/data/printer/expr/jsx.res b/tests/syntax_tests/data/printer/expr/jsx.res
index a76d07981c..87f04e2c66 100644
--- a/tests/syntax_tests/data/printer/expr/jsx.res
+++ b/tests/syntax_tests/data/printer/expr/jsx.res
@@ -546,4 +546,11 @@ let arrow_with_container_tag = el => <div>
   {t(nbsp ++ "(")}
   el
   {t(")")}
-</div>
\ No newline at end of file
+</div>
+
+<Component
+  prop1
+  // 
+  prop2>
+  {React.string("yow")}
+</Component>
\ No newline at end of file
diff --git a/tests/syntax_tests/data/printer/other/expected/signaturePicker.res.txt b/tests/syntax_tests/data/printer/other/expected/signaturePicker.res.txt
index f73d98b8cb..fe32b18e3c 100644
--- a/tests/syntax_tests/data/printer/other/expected/signaturePicker.res.txt
+++ b/tests/syntax_tests/data/printer/other/expected/signaturePicker.res.txt
@@ -28,7 +28,8 @@ let make = () => {
     <div className="pr-2 font-bold text-gray-400 text-lg"> {"Signature"->string} </div>
     <select
       id="country"
-      className="transition duration-150 ease-in-out sm:text-sm sm:leading-5 border-none font-bold text-2xl text-gray-600 bg-transparent">
+      className="transition duration-150 ease-in-out sm:text-sm sm:leading-5 border-none font-bold text-2xl text-gray-600 bg-transparent"
+    >
       {options
       ->Belt.List.map(option =>
         <option key={option->TimeSignature.toString}>
@@ -47,7 +48,8 @@ let make = () => {
       fill="none"
       strokeLinecap="round"
       strokeLinejoin="round"
-      className="text-gray-400">
+      className="text-gray-400"
+    >
       <polyline points="6 9 12 15 18 9" />
     </svg>
   </label>

From 31a5c287e210636f540025a3a9ecceb8a2ea3a7a Mon Sep 17 00:00:00 2001
From: nojaf <florian.verdonck@outlook.com>
Date: Mon, 25 Aug 2025 14:45:57 +0200
Subject: [PATCH 2/5] fmt of rescript code

---
 tests/gentype_tests/typescript-react-example/src/Hooks.res | 3 ++-
 tests/syntax_benchmarks/data/HeroGraphic.res               | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/tests/gentype_tests/typescript-react-example/src/Hooks.res b/tests/gentype_tests/typescript-react-example/src/Hooks.res
index 5468269e3a..0fb4c048b0 100644
--- a/tests/gentype_tests/typescript-react-example/src/Hooks.res
+++ b/tests/gentype_tests/typescript-react-example/src/Hooks.res
@@ -20,7 +20,8 @@ let make = (~vehicle) => {
       {React.string("child2")}
     </ImportHooks>
     <ImportHookDefault
-      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}>
+      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}
+    >
       {React.string("child1")}
       {React.string("child2")}
     </ImportHookDefault>
diff --git a/tests/syntax_benchmarks/data/HeroGraphic.res b/tests/syntax_benchmarks/data/HeroGraphic.res
index 22927a7fd1..9c0772682a 100644
--- a/tests/syntax_benchmarks/data/HeroGraphic.res
+++ b/tests/syntax_benchmarks/data/HeroGraphic.res
@@ -1997,7 +1997,8 @@ let make = (~width="760", ~height="380") =>
         y1="180.64"
         x2="362.765"
         y2="435.722"
-        gradientUnits="userSpaceOnUse">
+        gradientUnits="userSpaceOnUse"
+      >
         <stop offset="0.0658436" stopColor="#3A7DDD" />
         <stop offset="0.4001" stopColor="#265291" />
         <stop offset="0.571" stopColor="#1D3E6E" />

From 41eef4b4fc231cfc3b4a06c5d110d48ea3fb23ba Mon Sep 17 00:00:00 2001
From: nojaf <florian.verdonck@outlook.com>
Date: Fri, 29 Aug 2025 09:05:38 +0200
Subject: [PATCH 3/5] Add changelog entry

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index b5025db44b..61d24dde24 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -42,6 +42,7 @@
 - Autocomplete `@react.componentWithProps` attribute. https://github.com/rescript-lang/rescript/pull/7812
 - Add some missing iframe attributes to `domProps`. https://github.com/rescript-lang/rescript/pull/7813
 - Polish error message for inline record escaping scope. https://github.com/rescript-lang/rescript/pull/7808
+- Formatter: Change placement of closing `>` in JSX. https://github.com/rescript-lang/rescript/pull/7803
 
 #### :house: Internal
 

From b0716514229d7ba8132af3c62d6b58a4aacd9681 Mon Sep 17 00:00:00 2001
From: nojaf <florian.verdonck@outlook.com>
Date: Fri, 29 Aug 2025 09:12:54 +0200
Subject: [PATCH 4/5] Format ReScript code

---
 tests/gentype_tests/typescript-react-example/src/Hooks.res | 3 +--
 tests/syntax_benchmarks/data/HeroGraphic.res               | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/tests/gentype_tests/typescript-react-example/src/Hooks.res b/tests/gentype_tests/typescript-react-example/src/Hooks.res
index 0fb4c048b0..5468269e3a 100644
--- a/tests/gentype_tests/typescript-react-example/src/Hooks.res
+++ b/tests/gentype_tests/typescript-react-example/src/Hooks.res
@@ -20,8 +20,7 @@ let make = (~vehicle) => {
       {React.string("child2")}
     </ImportHooks>
     <ImportHookDefault
-      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}
-    >
+      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}>
       {React.string("child1")}
       {React.string("child2")}
     </ImportHookDefault>
diff --git a/tests/syntax_benchmarks/data/HeroGraphic.res b/tests/syntax_benchmarks/data/HeroGraphic.res
index 9c0772682a..22927a7fd1 100644
--- a/tests/syntax_benchmarks/data/HeroGraphic.res
+++ b/tests/syntax_benchmarks/data/HeroGraphic.res
@@ -1997,8 +1997,7 @@ let make = (~width="760", ~height="380") =>
         y1="180.64"
         x2="362.765"
         y2="435.722"
-        gradientUnits="userSpaceOnUse"
-      >
+        gradientUnits="userSpaceOnUse">
         <stop offset="0.0658436" stopColor="#3A7DDD" />
         <stop offset="0.4001" stopColor="#265291" />
         <stop offset="0.571" stopColor="#1D3E6E" />

From a80da0238c9288078ed88142f4ab799df2011d6f Mon Sep 17 00:00:00 2001
From: nojaf <florian.verdonck@outlook.com>
Date: Fri, 29 Aug 2025 09:18:40 +0200
Subject: [PATCH 5/5] I should have ran make build before make format

---
 tests/gentype_tests/typescript-react-example/src/Hooks.res | 3 ++-
 tests/syntax_benchmarks/data/HeroGraphic.res               | 3 ++-
 tests/tests/src/jsx_preserve_test.res                      | 3 ++-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/tests/gentype_tests/typescript-react-example/src/Hooks.res b/tests/gentype_tests/typescript-react-example/src/Hooks.res
index 5468269e3a..0fb4c048b0 100644
--- a/tests/gentype_tests/typescript-react-example/src/Hooks.res
+++ b/tests/gentype_tests/typescript-react-example/src/Hooks.res
@@ -20,7 +20,8 @@ let make = (~vehicle) => {
       {React.string("child2")}
     </ImportHooks>
     <ImportHookDefault
-      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}>
+      person={name: "DefaultImport", age: 42} renderMe={x => React.string(x["randomString"])}
+    >
       {React.string("child1")}
       {React.string("child2")}
     </ImportHookDefault>
diff --git a/tests/syntax_benchmarks/data/HeroGraphic.res b/tests/syntax_benchmarks/data/HeroGraphic.res
index 22927a7fd1..9c0772682a 100644
--- a/tests/syntax_benchmarks/data/HeroGraphic.res
+++ b/tests/syntax_benchmarks/data/HeroGraphic.res
@@ -1997,7 +1997,8 @@ let make = (~width="760", ~height="380") =>
         y1="180.64"
         x2="362.765"
         y2="435.722"
-        gradientUnits="userSpaceOnUse">
+        gradientUnits="userSpaceOnUse"
+      >
         <stop offset="0.0658436" stopColor="#3A7DDD" />
         <stop offset="0.4001" stopColor="#265291" />
         <stop offset="0.571" stopColor="#1D3E6E" />
diff --git a/tests/tests/src/jsx_preserve_test.res b/tests/tests/src/jsx_preserve_test.res
index 1046deb2e1..ff8635c1f2 100644
--- a/tests/tests/src/jsx_preserve_test.res
+++ b/tests/tests/src/jsx_preserve_test.res
@@ -150,5 +150,6 @@ let _youtube_iframe =
     frameBorder={0}
     allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture; web-share"
     referrerPolicy="strict-origin-when-cross-origin"
-    allowFullScreen={true}>
+    allowFullScreen={true}
+  >
   </iframe>