Merge pull request #2274 from automatisch/make-editabletypography-consistent

feat(EditableTypography): make icon size and position consistent

packages/web/src/components/EditableTypography/index.jsx

@@ -10,9 +10,8 @@ function EditableTypography(props) {
   const {
     children,
     onConfirm = noop,
-    iconPosition = 'start',
-    iconSize = 'large',
     sx,
+    iconColor = 'inherit',
     disabled = false,
     prefixValue = '',
     ...typographyProps
@@ -86,14 +85,10 @@ function EditableTypography(props) {
 
   return (
     <Box sx={sx} onClick={handleClick} editing={editing} disabled={disabled}>
-      {!disabled && iconPosition === 'start' && editing === false && (
-        <EditIcon fontSize={iconSize} sx={{ mr: 1 }} />
-      )}
-
       {component}
 
-      {!disabled && iconPosition === 'end' && editing === false && (
+      {!disabled && editing === false && (
-        <EditIcon fontSize={iconSize} sx={{ ml: 1 }} />
+        <EditIcon fontSize="small" color={iconColor} sx={{ ml: 1 }} />
       )}
     </Box>
   );
@@ -102,8 +97,7 @@ function EditableTypography(props) {
 EditableTypography.propTypes = {
   children: PropTypes.string.isRequired,
   disabled: PropTypes.bool,
-  iconPosition: PropTypes.oneOf(['start', 'end']),
+  iconColor: PropTypes.oneOf(['action', 'inherit']),
-  iconSize: PropTypes.oneOf(['small', 'large']),
   onConfirm: PropTypes.func,
   prefixValue: PropTypes.string,
   sx: PropTypes.object,

packages/web/src/components/EditorLayout/index.jsx

@@ -72,6 +72,7 @@ export default function EditorLayout() {
               variant="body1"
               onConfirm={onFlowNameUpdate}
               noWrap
+              iconColor="action"
               sx={{ display: 'flex', flex: 1, maxWidth: '50vw', ml: 2 }}
             >
               {flow?.name}