Skip to content
This repository has been archived by the owner on Apr 15, 2022. It is now read-only.

Switch Component #31

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Switch Component #31

wants to merge 11 commits into from

Conversation

phillip055
Copy link
Contributor

Added switch component using Headless UI

Todo:

  • Need to fix some css
  • Change name to Switch from Toggle

@vercel
Copy link

vercel bot commented Mar 8, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @Major-Eth on Vercel.

@Major-Eth first needs to authorize it.

@phillip055 phillip055 marked this pull request as draft March 8, 2022 15:45
@vercel
Copy link

vercel bot commented Mar 8, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/major-eth/yearn-web-lib/8S9NP8f3Ke7um1aCeofgRr7wdxMi
✅ Preview: Failed

@vercel vercel bot temporarily deployed to Preview March 8, 2022 15:50 Inactive
@Majorfi
Copy link
Contributor

Majorfi commented Mar 13, 2022

This pull request is being automatically deployed with Vercel (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/major-eth/yearn-web-lib/8S9NP8f3Ke7um1aCeofgRr7wdxMi ✅ Preview: Failed

Here is the deploy error :

src/components/Toggle/Toggle.tsx(61,14): error TS2769: No overload matches this call.
--
16:51:12.738 | Overload 1 of 2, '(props: { children?: ReactNode; color?: string \| undefined; slot?: string \| undefined; style?: CSSProperties \| undefined; title?: string \| undefined; ref?: ((instance: HTMLSpanElement \| null) => void) \| RefObject<...> \| null \| undefined; ... 254 more ...; onCheck: (enabled: boolean) => void; } & { ...; } & { ...; }): ReactElement<...>', gave the following error.
16:51:12.738 | Property 'onCheck' is missing in type '{ checked: boolean; "aria-hidden": "true"; height: number; width: number; }' but required in type '{ children?: ReactNode; color?: string \| undefined; slot?: string \| undefined; style?: CSSProperties \| undefined; title?: string \| undefined; ref?: ((instance: HTMLSpanElement \| null) => void) \| RefObject<...> \| null \| undefined; ... 254 more ...; onCheck: (enabled: boolean) => void; }'.
16:51:12.739 | Overload 2 of 2, '(props: StyledComponentPropsWithAs<"span", DefaultTheme, ToggleProps, never, "span", "span">): ReactElement<StyledComponentPropsWithAs<"span", DefaultTheme, ToggleProps, never, "span", "span">, string \| JSXElementConstructor<...>>', gave the following error.
16:51:12.739 | Property 'onCheck' is missing in type '{ checked: boolean; "aria-hidden": "true"; height: number; width: number; }' but required in type '{ children?: ReactNode; color?: string \| undefined; slot?: string \| undefined; style?: CSSProperties \| undefined; title?: string \| undefined; ref?: ((instance: HTMLSpanElement \| null) => void) \| RefObject<...> \| null \| undefined; ... 254 more ...; onCheck: (enabled: boolean) => void; }'.

@karelianpie
Copy link
Contributor

Hey @phillip055, any updates here? If you don't have the time I can try and wrap this up, please let me know

@phillip055
Copy link
Contributor Author

Hey @phillip055, any updates here? If you don't have the time I can try and wrap this up, please let me know
following this up!

@phillip055 phillip055 marked this pull request as ready for review March 26, 2022 05:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants