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

Add Form component #773

Merged
merged 105 commits into from
May 15, 2024
Merged

Add Form component #773

merged 105 commits into from
May 15, 2024

Conversation

KhudaDad414
Copy link
Member

@KhudaDad414 KhudaDad414 commented Aug 17, 2023

This PR introduces several enhancements to the @asyncapi/studio-ui package, specifically focusing on the form components.

Changes

  1. New Form Components: Added the following new components along with stories:

    • Form
    • Field
    • Dropdown
    • TextInput
    • TextArea
    • Fieldset
  2. Storybook Updates: Added support for tailwinds in stories for testing the components.

Related issue(s)
Resolves #752

Copy link
Member

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

In general, it looks amazing to me but found some gotchas.

Something I couldn't point in code is that the cursor of the DropdownMenu items is a text cursor (the one that looks like a capital i). It should be the arrow pointer (cursor-default class in Tailwind).

@KhudaDad414
Copy link
Member Author

@fmvilas the changes have been made.

princerajpoot20
princerajpoot20 previously approved these changes May 1, 2024
Amzani
Amzani previously approved these changes May 9, 2024
Copy link
Collaborator

@Amzani Amzani left a comment

Choose a reason for hiding this comment

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

LGTM

@KhudaDad414 KhudaDad414 dismissed stale reviews from Amzani and princerajpoot20 via 7ec312d May 15, 2024 10:23
Copy link

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Member

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

Ship it! 🚀

@KhudaDad414
Copy link
Member Author

/rtm

Copy link
Collaborator

@Amzani Amzani left a comment

Choose a reason for hiding this comment

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

LGTM

@KhudaDad414
Copy link
Member Author

/rtm

@asyncapi-bot asyncapi-bot merged commit 2c0ab6c into asyncapi:master May 15, 2024
27 checks passed
KhudaDad414 added a commit to KhudaDad414/studio that referenced this pull request Jun 4, 2024
Co-authored-by: samz <samir.amzani@gmail.com>
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.

Visual Designer: Form component
5 participants