Skip to content

Commit

Permalink
chore: can use aspect ratio value as string (#922)
Browse files Browse the repository at this point in the history
* feat: can use aspect ratio value as string

* test: add timeout seconds
  • Loading branch information
YimJiYoung authored Dec 28, 2023
1 parent fc56ec6 commit d60eb02
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/vibrant-components/src/lib/Drawer/Drawer.spec.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ describe('<Drawer />', () => {
});

it('content size is shrink', async () => {
await waitFor(() => expect(computedStyle.width).toEqual('480px'), { timeout: 2000 });
await waitFor(() => expect(computedStyle.width).toEqual('480px'), { timeout: 5000 });
});
});

Expand Down
2 changes: 1 addition & 1 deletion packages/vibrant-core/src/lib/props/sizing/sizing.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ const boxSizingProp = createSystemProp({

const aspectRatioProp = createSystemProp({
property: 'aspectRatio',
transform: (value: number) => ({
transform: (value: number | string) => ({
aspectRatio: value ? value.toString() : value,
}),
});
Expand Down
2 changes: 1 addition & 1 deletion packages/vibrant-core/src/lib/props/sizing/type.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@ export type SizingSystemProps = {
minHeight?: ResponsiveValue<number | string | 'auto' | 'fit-content' | 'max-content' | 'min-content'>;
maxHeight?: ResponsiveValue<number | string | 'auto' | 'fit-content' | 'max-content' | 'min-content' | 'none'>;
boxSizing?: 'border-box' | 'content-box';
aspectRatio?: ResponsiveValue<number>;
aspectRatio?: ResponsiveValue<number | string>;
};

1 comment on commit d60eb02

@vercel
Copy link

@vercel vercel bot commented on d60eb02 Dec 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YimJiYoung is attempting to deploy a commit to the Class101 Team on Vercel.

To accomplish this, @YimJiYoung needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Please sign in to comment.