From 38a36b894a33d5264d98c039fa90456b399423f3 Mon Sep 17 00:00:00 2001
From: MikelAlejoBR <mbarcina@redhat.com>
Date: Tue, 24 Mar 2026 10:07:14 -0400
Subject: [PATCH] fix: button is incorrectly disabled and style changes

There was a bug which made the "reset workspaces" button to be disabled
when the user was completely provisioned. It was due to a boolean
variable that should have been flipped, as it was disabling the button
when the user was ready, instead of doing it when it was NOT ready.

This commit also includes some style changes requested by the UX team.

SANDBOX-1067
---
 .../SandboxResetWorkspacesLink.tsx            | 25 +++++++++++++------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/plugins/sandbox/src/components/SandboxResetWorkspaces/SandboxResetWorkspacesLink.tsx b/plugins/sandbox/src/components/SandboxResetWorkspaces/SandboxResetWorkspacesLink.tsx
index f23b131..689fb15 100644
--- a/plugins/sandbox/src/components/SandboxResetWorkspaces/SandboxResetWorkspacesLink.tsx
+++ b/plugins/sandbox/src/components/SandboxResetWorkspaces/SandboxResetWorkspacesLink.tsx
@@ -2,6 +2,7 @@ import { alertApiRef, useApi } from '@backstage/core-plugin-api';
 import { RestartAlt } from '@mui/icons-material';
 import {
   Alert,
+  Box,
   Button,
   Dialog,
   DialogActions,
@@ -97,7 +98,7 @@ const SandboxResetLink = () => {
   return (
     <React.Fragment>
       <MenuItem
-        disabled={sandboxContext.userReady}
+        disabled={!sandboxContext.userReady}
         disableRipple
         disableTouchRipple
         onClick={handleMenuLinkClick}
@@ -113,7 +114,7 @@ const SandboxResetLink = () => {
         />
         <ListItemText
           primary="Reset workspaces"
-          secondary={sandboxContext.userReady && 'Only for active trials'}
+          secondary={!sandboxContext.userReady && 'Only for active trials'}
         />
       </MenuItem>
       <Dialog onClose={handleCancel} open={isModalOpen}>
@@ -134,22 +135,30 @@ const SandboxResetLink = () => {
             cannot undo this.
           </DialogContentText>
           {resetButtonState !== ResetButtonState.INITIAL && (
-            <Alert severity="warning" sx={{ marginTop: '25px' }}>
-              You are about to perform a destructive operation. Please make sure
-              that you want to do this.
-            </Alert>
+            <Box>
+              <Alert
+                severity="warning"
+                variant="standard"
+                sx={{
+                  marginTop: '25px',
+                }}
+              >
+                You are about to perform a destructive operation. Please make
+                sure that you want to do this.
+              </Alert>
+            </Box>
           )}
         </DialogContent>
         <DialogActions>
           <Button
             onClick={handleResetButtonClick}
-            variant="outlined"
+            variant="contained"
             color="error"
             disabled={resetButtonState === ResetButtonState.SUBMITTING}
           >
             {getResetButtonLabel(resetButtonState)}
           </Button>
-          <Button onClick={handleCancel} variant="contained" autoFocus>
+          <Button onClick={handleCancel} variant="outlined" autoFocus>
             Cancel
           </Button>
         </DialogActions>