Skip to content

Commit 5fadd5a

Browse files
committed
Refactor Copyable component to improve alignment of icons and structure of JSX elements
1 parent 47ffb7a commit 5fadd5a

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

app/components/common/Copyable.tsx

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@ export function Copyable({
2828

2929
function CopyIcon() {
3030
if (state === 'copy') {
31-
return <Copy className="align-text-top c-pointer" onClick={handleClick} size={13} />;
31+
return <Copy className="align-middle c-pointer" onClick={handleClick} size={13} />;
3232
} else if (state === 'copied') {
33-
return <CheckCircle className="align-text-top" size={13} />;
33+
return <CheckCircle className="align-middle" size={13} />;
3434
} else if (state === 'errored') {
3535
return (
3636
<span title="Please check your browser's copy permissions.">
37-
<XCircle className="align-text-top" size={13} />
37+
<XCircle className="align-middle" size={13} />
3838
</span>
3939
);
4040
}
@@ -53,29 +53,29 @@ export function Copyable({
5353

5454
function PrependCopyIcon() {
5555
return (
56-
<>
57-
<span className="font-size-tiny me-2">
56+
<div className="d-flex align-items-center">
57+
<div className="font-size-tiny me-2">
5858
<span className={textColor}>
5959
{message !== undefined && <span className="me-2">{message}</span>}
6060
<CopyIcon />
6161
</span>
62-
</span>
63-
{children}
64-
</>
62+
</div>
63+
<div>{children}</div>
64+
</div>
6565
);
6666
}
6767

6868
function ReplaceWithMessage() {
6969
return (
70-
<span className="d-flex flex-column flex-nowrap">
71-
<span className="font-size-tiny">
70+
<div className="d-flex flex-column flex-nowrap">
71+
<div className="font-size-tiny">
7272
<span className={textColor}>
7373
<CopyIcon />
7474
<span className="ms-2">{message}</span>
7575
</span>
76-
</span>
77-
<span className="v-hidden">{children}</span>
78-
</span>
76+
</div>
77+
<div className="v-hidden">{children}</div>
78+
</div>
7979
);
8080
}
8181

0 commit comments

Comments
 (0)