Skip to content

feat: button component + all variants + light / dark mode + storybook story#48

Merged
YounixM merged 4 commits intomainfrom
feat/button-component
Apr 25, 2025
Merged

feat: button component + all variants + light / dark mode + storybook story#48
YounixM merged 4 commits intomainfrom
feat/button-component

Conversation

@ahmadshaheer
Copy link
Contributor

@ahmadshaheer ahmadshaheer commented Apr 23, 2025

aks07
aks07 previously approved these changes Apr 24, 2025
Copy link
Contributor

@aks07 aks07 left a comment

Choose a reason for hiding this comment

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

one minor comment; can be taken up in subsequent PRs

@ahmadshaheer ahmadshaheer changed the base branch from feat/setup-mode-and-theme-support-using-tailwind-v4 to main April 24, 2025 09:37
@ahmadshaheer ahmadshaheer dismissed aks07’s stale review April 24, 2025 09:37

The base branch was changed.

@ahmadshaheer ahmadshaheer force-pushed the feat/button-component branch from aafdd53 to c54523d Compare April 24, 2025 12:33
@ahmadshaheer ahmadshaheer requested a review from aks07 April 24, 2025 14:12
@YounixM
Copy link
Member

YounixM commented Apr 24, 2025

@ahmadshaheer : As per the attached screenshots, the light mode and dark mode are inverted

@ahmadshaheer
Copy link
Contributor Author

@ahmadshaheer : As per the attached screenshots, the light mode and dark mode are inverted

@YounixM, the light/dark labels shown in the screenshot are to switch between light and dark mode. will improve the UI and make it similar to #36

aks07
aks07 previously approved these changes Apr 25, 2025
@YounixM YounixM merged commit 349d0ab into main Apr 25, 2025
2 of 3 checks passed
@YounixM YounixM deleted the feat/button-component branch April 25, 2025 12:10
@ahmadshaheer ahmadshaheer linked an issue Apr 27, 2025 that may be closed by this pull request
@ahmadshaheer ahmadshaheer removed a link to an issue Apr 27, 2025
@ahmadshaheer ahmadshaheer linked an issue Apr 27, 2025 that may be closed by this pull request
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.

Button

3 participants