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: updated divider component #99

Merged
merged 7 commits into from
Feb 23, 2024
Merged

Conversation

Samathingamajig
Copy link
Collaborator

@Samathingamajig Samathingamajig commented Feb 20, 2024

feat: updated divider component

image image

@Samathingamajig Samathingamajig changed the base branch from main to hackathon February 20, 2024 22:44
@Samathingamajig Samathingamajig changed the title sgunter/divider component feat: updated divider component Feb 20, 2024
@Longhorn-Developers Longhorn-Developers deleted a comment from github-actions bot Feb 20, 2024
@Longhorn-Developers Longhorn-Developers deleted a comment from github-actions bot Feb 20, 2024
src/stories/components/Button.stories.tsx Outdated Show resolved Hide resolved
unocss.config.ts Outdated Show resolved Hide resolved
src/stories/components/Divider.stories.tsx Outdated Show resolved Hide resolved
src/stories/components/Divider.stories.tsx Outdated Show resolved Hide resolved
src/stories/components/Divider.stories.tsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@doprz doprz left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@doprz doprz merged commit bb2efc0 into hackathon Feb 23, 2024
1 of 6 checks passed
@doprz doprz deleted the sgunter/divider-component branch February 23, 2024 03:24
doprz pushed a commit that referenced this pull request Mar 6, 2024
* feat: updated divider component

* refactor: inlined Divider's classes, simplified stories

* fix: style to unocss in story

* style: renamed variant to orientation for buttons

* docs: updated comments in Divider after prop name change
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.

3 participants