Buttons audit

AntonelliLab · Feb 17, 2025 · b109f17 · b109f17
1 parent 1712a52
commit b109f17
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 8 deletions.
diff --git a/src/app/CitationModal.js b/src/app/CitationModal.js
@@ -77,6 +77,7 @@ function CitationModal({ citation }) {
           <Button
             aria-label="Copy to clipboard"
             variant="contained"
+            color="secondary"
             onClick={citation.copyToClipboard}
             style={{ marginRight: 10 }}
           >

diff --git a/src/app/PartitionEditor.js b/src/app/PartitionEditor.js
@@ -221,6 +221,7 @@ function PartitionOnCard({ alignment }) {
         <Button
           aria-label="Save"
           variant="contained"
+          color="secondary"
           disabled={!hasChange}
           onClick={onClickSave}
         >

diff --git a/src/app/components/ErrorDialog.js b/src/app/components/ErrorDialog.js
@@ -52,23 +52,36 @@ export default function ErrorDialog({ error, onClose, needReload, title }) {
 
   const CloseAction = (
     <DialogActions>
-      <Button onClick={closeHandler} variant="contained" color="primary">
+      <Button onClick={closeHandler} variant="outlined">
         {closeMessage}
       </Button>
     </DialogActions>
   );
 
-  const Actions = reported ? CloseAction : (
+  const Actions = reported ? (
+    CloseAction
+  ) : (
     <DialogActions>
-      <Button onClick={closeHandler} color="secondary">
-        { closeMessage }
-      </Button>
-      <Button href={mailtoContent} onClick={handleReportToMail} variant="contained" color="primary" autoFocus>
+      <Button
+        href={mailtoContent}
+        onClick={handleReportToMail}
+        variant="contained"
+        color="secondary"
+        autoFocus
+      >
         Report issue on mail
       </Button>
-      <Button onClick={handleReportToGithub} variant="contained" color="primary" autoFocus>
+      <Button
+        onClick={handleReportToGithub}
+        variant="contained"
+        color="secondary"
+        autoFocus
+      >
         Report issue on GitHub
       </Button>
+      <Button onClick={closeHandler} variant="outlined">
+        {closeMessage}
+      </Button>
     </DialogActions>
   );
 

diff --git a/src/app/components/ModifiedDialog.js b/src/app/components/ModifiedDialog.js
@@ -45,7 +45,7 @@ export default function ModifiedDialog({ show, onClose, messages }) {
         </Box>
       </DialogContent>
       <DialogActions>
-        <Button onClick={onClose} variant="contained" color="primary">
+        <Button onClick={onClose} variant="outlined">
           Close
         </Button>
       </DialogActions>