Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Generic Embed Component #180

Merged
merged 4 commits into from
May 5, 2022

Conversation

Cahllagerfeld
Copy link
Contributor

@Cahllagerfeld Cahllagerfeld commented May 4, 2022

This PR is based on #178 and adds a generic component to the library.

I added the Component to the Index Page as well and turned it into a Spotify Version. I tried to document how the component works inside a comment within the code preview, not sure if that's how it's supposed to be though.

Feel free to give some feedback on this 👍

@vercel
Copy link

vercel bot commented May 4, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
sveltekit-embed ✅ Ready (Inspect) Visit Preview May 5, 2022 at 1:35PM (UTC)

@spences10
Copy link
Owner

Hey @Cahllagerfeld, thanks. So, couple of things, I think the component name as <Generic> could be a bit more descriptive, <GenericEmbed> maybe?

The use of the props is something I'll need to do something with, maybe add a component to take care of the props and usage.

@spences10
Copy link
Owner

Thanks man!

I've just moved it over to MDSvex because I wanted to add usage and that was the best way I could come up with 😊

I'll be keeping my eye on https://github.com/leveluptuts/bookit though!

@spences10 spences10 merged commit be417c1 into spences10:main May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants