From 6ca6299e085dae028fb212fcc1610849bdb76da0 Mon Sep 17 00:00:00 2001
From: Marcus <marcustyphoon@gmail.com>
Date: Wed, 29 May 2024 19:15:11 -0700
Subject: [PATCH] simplify "pinned" / "starred" language in UI

---
 .../results/SavedResultManager/SavedResultManager.jsx |  4 ++--
 src/components/sections/results/table/ResultTable.jsx | 11 +++--------
 .../sections/results/table/ResultTableHeaderRow.jsx   |  5 +----
 .../sections/results/table/ResultTableRow.jsx         |  8 ++++----
 4 files changed, 10 insertions(+), 18 deletions(-)

diff --git a/src/components/sections/results/SavedResultManager/SavedResultManager.jsx b/src/components/sections/results/SavedResultManager/SavedResultManager.jsx
index 510d6be04..f6a91699f 100644
--- a/src/components/sections/results/SavedResultManager/SavedResultManager.jsx
+++ b/src/components/sections/results/SavedResultManager/SavedResultManager.jsx
@@ -215,7 +215,7 @@ export default function SavedResultManager({ isOpen, setOpen }) {
 
       <DialogContent sx={{ padding: 2 }} dividers>
         <Typography fontWeight={200}>
-          <Trans>Temporary saved builds</Trans>{' '}
+          <Trans>Current pinned builds</Trans>{' '}
           <HelperIcon
             text={t('These builds will be deleted after you leave or refresh this page.')}
             size="small"
@@ -343,7 +343,7 @@ export default function SavedResultManager({ isOpen, setOpen }) {
                       </IconButton>
                     </Tooltip>
 
-                    <Tooltip title={t('Copy build to temporary saved builds')}>
+                    <Tooltip title={t('Load as pinned build')}>
                       <IconButton onClick={handleCopyToTemporary(character)}>
                         <ArrowCircleUpIcon fontSize="small" />
                       </IconButton>
diff --git a/src/components/sections/results/table/ResultTable.jsx b/src/components/sections/results/table/ResultTable.jsx
index c3eda6341..9e0ba7a51 100644
--- a/src/components/sections/results/table/ResultTable.jsx
+++ b/src/components/sections/results/table/ResultTable.jsx
@@ -1,5 +1,4 @@
-import { HelperIcon } from '@discretize/react-discretize-components';
-import ManageAccountsIcon from '@mui/icons-material/ManageAccounts';
+import SaveAsIcon from '@mui/icons-material/SaveAs';
 import { Box, IconButton, Typography } from '@mui/material';
 import Table from '@mui/material/Table';
 import TableBody from '@mui/material/TableBody';
@@ -220,15 +219,11 @@ const StickyHeadTable = () => {
 
       <Box display="flex" alignItems="center" className={classes.tablehead}>
         <Typography flexGrow={1} ml={2} fontWeight={600} fontFamily="Raleway">
-          <Trans>Saved Results</Trans>{' '}
-          <HelperIcon
-            text={t('Click the star icon to save a result for comparison.')}
-            fontSize="1rem"
-          />
+          <Trans>Pinned Results</Trans>{' '}
         </Typography>
 
         <IconButton size="small" sx={{ margin: 1 }} onClick={() => setManagerOpen(true)}>
-          <ManageAccountsIcon />
+          <SaveAsIcon />
         </IconButton>
         <SavedResultManager isOpen={managerOpen} setOpen={setManagerOpen} />
       </Box>
diff --git a/src/components/sections/results/table/ResultTableHeaderRow.jsx b/src/components/sections/results/table/ResultTableHeaderRow.jsx
index e99044a5b..99afb2fe9 100644
--- a/src/components/sections/results/table/ResultTableHeaderRow.jsx
+++ b/src/components/sections/results/table/ResultTableHeaderRow.jsx
@@ -47,10 +47,7 @@ const ResultTableHeaderRow = ({
   return (
     <TableRow>
       <TableCell className={classes.tablehead} align="center" padding="none">
-        <HelperIcon
-          text={t('Click the star icon to save a result for comparison.')}
-          fontSize="1rem"
-        />
+        <HelperIcon text={t('Pin results for comparison between calculations.')} fontSize="1rem" />
       </TableCell>
       <TableCell className={classes.tablehead}>
         {t('priorityGoal', {
diff --git a/src/components/sections/results/table/ResultTableRow.jsx b/src/components/sections/results/table/ResultTableRow.jsx
index 86c1df4da..ffec34e3b 100644
--- a/src/components/sections/results/table/ResultTableRow.jsx
+++ b/src/components/sections/results/table/ResultTableRow.jsx
@@ -1,6 +1,6 @@
 import { Item, Profession } from '@discretize/gw2-ui-new';
 import CloseIcon from '@mui/icons-material/Close';
-import StarRoundedIcon from '@mui/icons-material/StarRounded';
+import PushPinIcon from '@mui/icons-material/PushPin';
 import { Typography } from '@mui/material';
 import TableCell from '@mui/material/TableCell';
 import TableRow from '@mui/material/TableRow';
@@ -82,17 +82,17 @@ const ResultTableRow = ({
             }}
           />
         ) : (
-          <StarRoundedIcon
+          <PushPinIcon
             sx={
               saved
                 ? {
-                    color: 'star',
+                    color: 'primary.main',
                   }
                 : {
                     opacity: '0.2',
                     '&:hover': {
                       opacity: '1',
-                      color: 'star',
+                      color: 'primary.main',
                     },
                   }
             }