Skip to content

feat: button story Components and Component reusability #809

Merged
asyncapi-bot merged 21 commits into
asyncapi:masterfrom
TenzDelek:buttonstory
Jan 8, 2026
Merged

feat: button story Components and Component reusability #809
asyncapi-bot merged 21 commits into
asyncapi:masterfrom
TenzDelek:buttonstory

Conversation

@TenzDelek

Copy link
Copy Markdown
Member

Description
image

this pr introduces the button story. on top of that we have made changes in the existing component to use the Button component instead of the direct button from the native.
we also introduced a new prop inspire by the main website button component where we take in text and icon as a prop.

  • audit repository to reuse the Button component
  • introduce prop for icon and text
  • storybook creation for button

and one file changes if of download icon which is a left over of previous pr #802

Related issue(s)

GSOC

@netlify

netlify Bot commented Sep 27, 2025

Copy link
Copy Markdown

Deploy Preview for peaceful-ramanujan-288045 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit e58fb43
🔍 Latest deploy log https://app.netlify.com/projects/peaceful-ramanujan-288045/deploys/695dea540b6bdb000889269f
😎 Deploy Preview https://deploy-preview-809--peaceful-ramanujan-288045.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 project configuration.

@TenzDelek TenzDelek changed the title feat: Button story Components feat: button story Components and Component reusability Sep 27, 2025
Comment thread components/Buttons/button.stories.tsx Outdated
Comment thread components/Buttons/button.stories.tsx Outdated
@AceTheCreator

Copy link
Copy Markdown
Member

The center icon looks a bit off as a button. It’s fine to wrap it in a div and style it using an anchor tag instead; it doesn’t really need to be a button element.

AsyncAPI-Conference-10-10-2025_02_49_PM

@TenzDelek

Copy link
Copy Markdown
Member Author

thank you for review @AceTheCreator , will make the changes as stated

@TenzDelek

Copy link
Copy Markdown
Member Author

@AceTheCreator @ashmit-coder ready for review

@AceTheCreator AceTheCreator left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Left you some final comment @TenzDelek

Comment thread components/Buttons/button.stories.tsx Outdated
Comment thread components/Form/Cfp/stepFour.tsx Outdated
@TenzDelek
TenzDelek marked this pull request as draft December 21, 2025 13:22
@TenzDelek
TenzDelek marked this pull request as ready for review December 21, 2025 13:34
@TenzDelek

Copy link
Copy Markdown
Member Author

sorry for the delay. pinging for review @AceTheCreator @ashmit-coder

@AceTheCreator AceTheCreator left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@AceTheCreator

Copy link
Copy Markdown
Member

/rtm

@asyncapi-bot
asyncapi-bot merged commit 8dda75c into asyncapi:master Jan 8, 2026
14 checks passed
@TenzDelek
TenzDelek deleted the buttonstory branch January 8, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants