From d32fddc286720d962dbb0fa225bd9a4197be4103 Mon Sep 17 00:00:00 2001
From: David Crespo <david.crespo@oxidecomputer.com>
Date: Fri, 26 Jul 2024 11:30:53 -0500
Subject: [PATCH] Revert "Focus confirm button instead of cancel in modals
 (#2340)"

6615cb6b5b2470cdc0ae6d66392115c413e0adc8
---
 app/ui/lib/Modal.tsx | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/app/ui/lib/Modal.tsx b/app/ui/lib/Modal.tsx
index 8e61d0f61..3994420c0 100644
--- a/app/ui/lib/Modal.tsx
+++ b/app/ui/lib/Modal.tsx
@@ -132,9 +132,10 @@ Modal.Footer = ({
 }) => (
   <footer className="flex items-center justify-between border-t px-3 py-3 border-secondary">
     <div className="mr-4">{children}</div>
-    <div className="flex flex-row-reverse gap-2">
-      {/* Note the confirm button is first so it autofocuses when the modal opens,
-          but it displays in the right order because of flex-row-reverse */}
+    <div className="space-x-2">
+      <Button variant="secondary" size="sm" onClick={onDismiss}>
+        {cancelText || 'Cancel'}
+      </Button>
       <Button
         size="sm"
         variant={actionType}
@@ -144,9 +145,6 @@ Modal.Footer = ({
       >
         {actionText}
       </Button>
-      <Button variant="secondary" size="sm" onClick={onDismiss}>
-        {cancelText || 'Cancel'}
-      </Button>
     </div>
   </footer>
 )