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(CustomSelect): Upstream CustomSelect component [WD-17720] #1168

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mas-who
Copy link
Contributor

@mas-who mas-who commented Jan 6, 2025

Done

  • Upstream CustomSelect component from lxd-ui

QA

Storybook

To see rendered examples of all react-components, run:

yarn start

QA in your project

from react-components run:

yarn build
npm pack

Install the resulting tarball in your project with:

yarn add <path-to-tarball>

QA steps

  • Check selecting an option
  • Check search functionality
  • Check height adjustment against viewport size
  • Check keyboard navigation

@webteam-app
Copy link

@mas-who mas-who changed the title Upstream CustomSelect component [WD-17720] feat(CustomSelect): Upstream CustomSelect component [WD-17720] Jan 6, 2025
@mas-who mas-who force-pushed the upstream-custom-select branch 3 times, most recently from 6cb56e0 to 8b62796 Compare January 6, 2025 18:58
@mas-who mas-who force-pushed the upstream-custom-select branch from 8b62796 to 28fa618 Compare January 7, 2025 20:13
@mas-who mas-who marked this pull request as ready for review January 7, 2025 20:16
Copy link
Contributor

@fasih-mehmood fasih-mehmood left a comment

Choose a reason for hiding this comment

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

Great work @mas-who, left a couple of comments. This looks and works great, thank you!

@mas-who mas-who force-pushed the upstream-custom-select branch from 4b0a3e4 to a08f5a0 Compare January 8, 2025 08:03
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