Skip to content

Commit 05f3ec0

Browse files
authored
Fix: Icons cant be on scale (#3689)
* icons cant be on scale * whoops thats not supposed to be there * fix story size
1 parent fabae3d commit 05f3ec0

File tree

6 files changed

+10
-14
lines changed

6 files changed

+10
-14
lines changed

packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/Comments/Comment.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ export const Comment = React.memo(({ comment }: any) => {
7070
<Icon name="check" title="Resolved" color="green" />
7171
)}
7272
<Menu>
73-
<Menu.IconButton name="more" title="Comment actions" size={3} />
73+
<Menu.IconButton name="more" title="Comment actions" size={12} />
7474
<Menu.List>
7575
<Menu.Item
7676
onSelect={() =>

packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/Comments/Dialog/Reply.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ export const Reply = ({
5757
{state.user.id === author.id && (
5858
<Stack align="center">
5959
<Menu>
60-
<Menu.IconButton name="more" title="Reply actions" size={3} />
60+
<Menu.IconButton name="more" title="Reply actions" size={12} />
6161
<Menu.List>
6262
<Menu.Item
6363
onSelect={() =>

packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/Comments/Dialog/index.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ export const Dialog = props => {
9696
})
9797
}
9898
name="check"
99-
size={4}
99+
size={14}
100100
title="Resolve Comment"
101101
css={css({
102102
transition: 'color',
@@ -106,7 +106,7 @@ export const Dialog = props => {
106106
/>
107107
<IconButton
108108
name="cross"
109-
size={3}
109+
size={10}
110110
title="Close comment dialog"
111111
onClick={closeDialog}
112112
/>
@@ -146,7 +146,7 @@ export const Dialog = props => {
146146
<Menu.IconButton
147147
name="more"
148148
title="Comment actions"
149-
size={3}
149+
size={12}
150150
/>
151151
<Menu.List>
152152
<Menu.Item

packages/app/src/app/pages/Sandbox/Editor/Workspace/screens/Comments/index.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ export const Comments: React.FC = () => {
4949
>
5050
<Icon
5151
name="comments"
52-
size={20}
52+
size={80}
5353
color="mutedForeground"
5454
css={{ opacity: 0.2 }}
5555
/>
@@ -87,7 +87,7 @@ export const Comments: React.FC = () => {
8787
className="icon-button"
8888
name="filter"
8989
title="Filter comments"
90-
size={3}
90+
size={12}
9191
/>
9292
<Menu.List>
9393
{options.map(option => (

packages/components/src/components/Icon/index.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ export const Simple = () => (
1212
<Stack gap={2}>
1313
<Icon name="github" />
1414
<Icon name="github" css={{ color: 'blues.500' }} />
15-
<Icon name="github" size={6} />
15+
<Icon name="github" size={24} />
1616
</Stack>
1717
);
1818

packages/components/src/components/Icon/index.tsx

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,11 @@ type IconProps = React.SVGAttributes<SVGElement> &
1818

1919
export const Icon: React.FC<IconProps> = ({
2020
name = 'notFound',
21-
size = 4,
21+
size = 16,
2222
color = 'inherit',
2323
...props
2424
}) => {
2525
const SVG = icons[name];
26-
// we use a 4px scale for space and size
27-
const scaledSize = size * 4;
2826

29-
return (
30-
<SVG width={scaledSize} height={scaledSize} color={color} {...props} />
31-
);
27+
return <SVG width={size} height={size} color={color} {...props} />;
3228
};

0 commit comments

Comments
 (0)