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

sveltekit components #455

Merged
merged 18 commits into from
Feb 28, 2024
Merged

sveltekit components #455

merged 18 commits into from
Feb 28, 2024

Conversation

eyeseast
Copy link
Collaborator

@eyeseast eyeseast commented Feb 27, 2024

  • Button
  • Checkbox
  • PageToolbar

Copy link

netlify bot commented Feb 27, 2024

Deploy Preview for documentcloud-frontend ready!

Name Link
🔨 Latest commit e4129a4
🔍 Latest deploy log https://app.netlify.com/sites/documentcloud-frontend/deploys/65df451bccf6bb00082f6747
😎 Deploy Preview https://deploy-preview-455.muckcloud.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@eyeseast eyeseast marked this pull request as ready for review February 27, 2024 21:22
@eyeseast eyeseast requested a review from allanlasser February 27, 2024 21:23
Copy link
Member

@allanlasser allanlasser left a comment

Choose a reason for hiding this comment

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

Looking good so far! Lots of little feedbacks for lots of little components.

Also while Chromatic check is passing, the build is failing for some reason—happy to look into it!

src/lib/components/common/Checkbox.svelte Outdated Show resolved Hide resolved
src/lib/components/common/Checkbox.svelte Outdated Show resolved Hide resolved
src/lib/components/common/PageToolbar.svelte Show resolved Hide resolved
src/lib/components/common/Button.svelte Show resolved Hide resolved
src/lib/components/common/Button.svelte Show resolved Hide resolved
src/lib/components/common/stories/Button.stories.svelte Outdated Show resolved Hide resolved
src/lib/components/common/Button.svelte Show resolved Hide resolved
src/lib/components/common/Button.svelte Show resolved Hide resolved
src/lib/components/common/Button.svelte Outdated Show resolved Hide resolved
@eyeseast
Copy link
Collaborator Author

Looking good so far! Lots of little feedbacks for lots of little components.

Also while Chromatic check is passing, the build is failing for some reason—happy to look into it!

The issue is Svue. All the current site components that touch Svue in any way throw errors if Storybook tries to render them using Vite/Rollup. Storybook still works for newer/stateless components, and still deploys, but it's broken in some parts.

Copy link
Member

@allanlasser allanlasser left a comment

Choose a reason for hiding this comment

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

Caught three last small things. After that, merge away!

src/lib/components/common/Button.svelte Show resolved Hide resolved
src/lib/components/common/Checkbox.svelte Show resolved Hide resolved
src/lib/components/common/PageToolbar.svelte Show resolved Hide resolved
Copy link
Member

@allanlasser allanlasser left a comment

Choose a reason for hiding this comment

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

Caught three last small things. After that, merge away!

@eyeseast eyeseast merged commit 390a31a into sveltekit Feb 28, 2024
6 checks passed
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.

2 participants