Skip to content

Commit 25f0169

Browse files
committed
design feedback
1 parent 4d59b14 commit 25f0169

File tree

2 files changed

+16
-4
lines changed

2 files changed

+16
-4
lines changed

src/pages/spot/QuickButtons.tsx

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ export const QuickButtons = ({
8484
type="button"
8585
onClick={isEditing ? handleConfirmEdit : handleEdit}
8686
disabled={disabled}
87+
isEditing={isEditing}
8788
>
8889
<Icon iconName={isEditing ? IconName.Check : IconName.Pencil2} size="1rem" />
8990
</$QuickButtonContainer>
@@ -127,7 +128,7 @@ const QuickButtonContainerStyles = css`
127128
border: 1px solid var(--color-layer-4);
128129
padding: 0 0.5rem;
129130
font: var(--font-base-medium);
130-
color: var(--color-text);
131+
color: var(--color-text-1);
131132
min-width: 0;
132133
133134
@media (prefers-reduced-motion: no-preference) {
@@ -142,10 +143,12 @@ const $InputQuickButtonContainer = styled.div`
142143
143144
&:focus-within {
144145
background-color: var(--color-layer-4);
146+
border-color: var(--color-layer-5);
147+
color: var(--color-text-2);
145148
}
146149
`;
147150

148-
const $QuickButtonContainer = styled.button<{ isSelected?: boolean }>`
151+
const $QuickButtonContainer = styled.button<{ isSelected?: boolean; isEditing?: boolean }>`
149152
${QuickButtonContainerStyles}
150153
151154
${({ isSelected }) =>
@@ -154,6 +157,14 @@ const $QuickButtonContainer = styled.button<{ isSelected?: boolean }>`
154157
background-color: var(--color-layer-4);
155158
`}
156159
160+
${({ isEditing }) =>
161+
isEditing &&
162+
css`
163+
background-color: var(--color-accent-faded);
164+
color: var(--color-accent);
165+
border-color: var(--color-accent);
166+
`}
167+
157168
@media (hover: hover) {
158169
&:hover {
159170
background-color: var(--color-layer-3);

src/pages/spot/SpotTabs.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,9 +79,10 @@ const $Root = styled(Root)`
7979
const $List = styled(List)`
8080
display: flex;
8181
flex-direction: row;
82-
padding: 3px;
82+
padding: 2px;
8383
background-color: var(--color-layer-1);
8484
border-radius: var(--tab-border-radius);
85+
gap: 2px;
8586
`;
8687

8788
const spotTabVariants: Record<SpotTabVariant, ReturnType<typeof css>> = {
@@ -125,7 +126,7 @@ const $Trigger = styled(Trigger)<{
125126
font-weight: 500;
126127
flex: 1;
127128
height: 37px;
128-
border-radius: var(--tab-border-radius);
129+
border-radius: calc(var(--tab-border-radius) - 2px);
129130
padding: 0 1rem;
130131
text-align: center;
131132
`;

0 commit comments

Comments
 (0)