From 38798bfab96129ac6e413fcfb576167fc32192bc Mon Sep 17 00:00:00 2001
From: "Leonardo R. Dias" <47978193+leoreisdias@users.noreply.github.com>
Date: Fri, 11 Sep 2026 19:10:58 -0300
Subject: [PATCH] fix(review): prevent submission footer overlap

---
 .../review-editor/components/ReviewSubmissionDialog.tsx  | 5 +++--
 .../components/ReviewSubmissionDialog.ui.test.tsx        | 9 +++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/packages/review-editor/components/ReviewSubmissionDialog.tsx b/packages/review-editor/components/ReviewSubmissionDialog.tsx
index bb8f1ce44..a80149f5c 100644
--- a/packages/review-editor/components/ReviewSubmissionDialog.tsx
+++ b/packages/review-editor/components/ReviewSubmissionDialog.tsx
@@ -551,8 +551,10 @@ export function ReviewSubmissionDialog({
           View on {platformLabel} after submitting
         </label>
 
+        </div>
+
         {/* Actions */}
-        <div className="sticky bottom-0 -mx-4 -mb-4 flex justify-end gap-2 border-t border-border/50 bg-card px-4 pb-4 pt-3 sm:-mx-6 sm:-mb-6 sm:px-6 sm:pb-6">
+        <div className="shrink-0 flex justify-end gap-2 border-t border-border/50 bg-card px-4 pb-4 pt-3 sm:px-6 sm:pb-6">
           <button
             data-pn-touch-target
             onClick={onCancel}
@@ -586,7 +588,6 @@ export function ReviewSubmissionDialog({
                     : 'Post Comments'}
           </button>
         </div>
-        </div>
       </DialogContent>
     </Dialog>
   );
diff --git a/packages/review-editor/components/ReviewSubmissionDialog.ui.test.tsx b/packages/review-editor/components/ReviewSubmissionDialog.ui.test.tsx
index dbc69af7b..326ece0a8 100644
--- a/packages/review-editor/components/ReviewSubmissionDialog.ui.test.tsx
+++ b/packages/review-editor/components/ReviewSubmissionDialog.ui.test.tsx
@@ -90,6 +90,15 @@ describe('ReviewSubmissionDialog submission outcomes', () => {
     expect(actionButton()?.hasAttribute('data-pn-touch-target')).toBe(true);
   });
 
+  test.skipIf(!hasDom)('keeps the action footer outside the scrollable form body', async () => {
+    await renderSubmission({ targets: [baseTarget], orphans: [] });
+
+    const scrollableBody = document.querySelector('textarea')?.closest('.overflow-y-auto');
+
+    expect(scrollableBody).not.toBeNull();
+    expect(scrollableBody?.contains(actionButton())).toBe(false);
+  });
+
   test.skipIf(!hasDom)('ignores Escape while a platform submission is in flight', async () => {
     let cancelCount = 0;
     await renderSubmission(