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: added drawer component #2223

Closed
wants to merge 1 commit into from
Closed

Conversation

1111mp
Copy link
Contributor

@1111mp 1111mp commented Jan 10, 2024

📝 Description

Added Drawer component.

Learned about some user feature requests from #2134 [Feature Request] Some Components Feature Suggestions, so I started to implement them from the simplest components (Because it can almost reuse Modal’s design and code).

⛳️ Current behavior (updates)

There is currently no support for the Drawer component.

🚀 New behavior

We can have the Drawer component.

💣 Is this a breaking change (Yes/No):

No. Supports the new Drawer component, which will not have any impact on other components.

📝 Additional Information

Screenshot 2024-01-10 at 18 26 54

There is currently some work yet to be completed, such as Drawer documentation and usage examples. Because I don’t know whether such code complies with the specifications of the warehouse, I initiated a PR in advance to confirm this. If all goes well I will continue with the rest of the work.

Feel free to point out my questions at any time. Of course, if you already have Drawer-related implementation plans or other questions, please let me know and I will close this PR. Thank you.

This is a continuation of the previous PR #2202.

I have read the contributing guidelines and code of conduct document.

Signed-off-by: The1111mp <The1111mp@outlook.com>
Copy link

changeset-bot bot commented Jan 10, 2024

⚠️ No Changeset found

Latest commit: efbc9ab

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Jan 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nextui-storybook-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2024 10:34am

Copy link

vercel bot commented Jan 10, 2024

@1111mp is attempting to deploy a commit to the NextUI Team on Vercel.

A member of the Team first needs to authorize it.

@vitalijalbu
Copy link

Hi, any updates on this PR ?? People are waiting... :D

@samuelbohl
Copy link

any updates on this?

@wingkwong wingkwong changed the base branch from main to canary April 28, 2024 14:55
@Mingyang-Li
Copy link

Component looks great 🤩

@jenslys
Copy link

jenslys commented Aug 25, 2024

any updates on this?

@ohIcee
Copy link

ohIcee commented Aug 29, 2024

Any updates?

@wingkwong wingkwong added this to the v2.5.0 milestone Sep 8, 2024
@wingkwong wingkwong self-assigned this Sep 8, 2024
@wingkwong
Copy link
Member

@1111mp can you ping me at discord?

@jrgarciadev jrgarciadev mentioned this pull request Nov 4, 2024
@jrgarciadev
Copy link
Member

Hey @1111mp thanks for your contribution, we will handle this internally here #3986

@jrgarciadev jrgarciadev closed this Nov 4, 2024
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.

[Feature Request] Need A Drawer Component [Feature Request] Drawer Component
8 participants