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

feat(card): add large size and space option to footer #312

Merged
merged 3 commits into from
Jul 19, 2023

Conversation

kiaking
Copy link
Member

@kiaking kiaking commented Jul 12, 2023

Add large option to <SCard>. Only relevant when using inside <SModal>.

Also add :space option to <SCradFooter> so that it can have the same spacing with <SCardBlock> component. This spacing is needed to align with some of our app design.

Also, improved docs a bit.

@kiaking kiaking added the enhancement New feature or request label Jul 12, 2023
@kiaking kiaking self-assigned this Jul 12, 2023
@netlify
Copy link

netlify bot commented Jul 12, 2023

Deploy Preview for sefirot-story ready!

Name Link
🔨 Latest commit 127666f
🔍 Latest deploy log https://app.netlify.com/sites/sefirot-story/deploys/64b66b9be4e86400081cd934
😎 Deploy Preview https://deploy-preview-312--sefirot-story.netlify.app
📱 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.

@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02 🎉

Comparison is base (4546cab) 83.57% compared to head (127666f) 83.60%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #312      +/-   ##
==========================================
+ Coverage   83.57%   83.60%   +0.02%     
==========================================
  Files         123      123              
  Lines       10515    10532      +17     
  Branches      448      448              
==========================================
+ Hits         8788     8805      +17     
  Misses       1727     1727              
Impacted Files Coverage Δ
lib/components/SCard.vue 100.00% <100.00%> (ø)
lib/components/SCardBlock.vue 100.00% <100.00%> (ø)
lib/components/SCardFooter.vue 100.00% <100.00%> (ø)
lib/components/SCardFooterActions.vue 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@netlify
Copy link

netlify bot commented Jul 12, 2023

Deploy Preview for sefirot-docs ready!

Name Link
🔨 Latest commit 127666f
🔍 Latest deploy log https://app.netlify.com/sites/sefirot-docs/deploys/64b66b9b8c8b5d00076c4caa
😎 Deploy Preview https://deploy-preview-312--sefirot-docs.netlify.app
📱 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.

docs/components/card.md Outdated Show resolved Hide resolved
docs/components/card.md Outdated Show resolved Hide resolved
kiaking and others added 2 commits July 18, 2023 19:37
Co-authored-by: Cue <[email protected]>
Co-authored-by: Cue <[email protected]>
@kiaking kiaking merged commit f628b8e into main Jul 19, 2023
@kiaking kiaking deleted the card-add-more-size-variants branch July 19, 2023 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants