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

fix(select): dropdown zindex added #1691

Merged
merged 7 commits into from
Mar 26, 2021
Merged

fix(select): dropdown zindex added #1691

merged 7 commits into from
Mar 26, 2021

Conversation

EdwardIrby
Copy link
Contributor

No description provided.

@KaidenR
Copy link
Contributor

KaidenR commented Mar 25, 2021

This could cause an issue if a dialog needs to show while a Select menu is open on the page underneath it. But hopefully menus will always be closed when dialogs open so maybe it's okay.

- Move item spacing from line-height into padding for better spacing when item has multiple lines
- Made item and item w/description consistent by inheriting name styles from option
- Increases spacing betweeen name and description
- Added story to show items next to items w/description
@EdwardIrby EdwardIrby merged commit 19b03ad into master Mar 26, 2021
@EdwardIrby EdwardIrby deleted the fix/select branch March 26, 2021 16:32
jaketrent added a commit that referenced this pull request Apr 7, 2021
* master:
  build: publish
  fix(datepicker): closes on click outside resolves #1655 (#1700)
  tagsinput: fix scroll bar position when items/pills overflow  (#1702)
  flexibility: remove base from color scale & add parseToRgb (#1696)
  fix(searchinput): weird type bug resolved (#1698)
  feat(core): temp core util for refactoring to vanillia css (#1689)
  build: publish
  select: stop intercepting all key events (#1699)
  build: publish
  fix(storybook-addon-theme): update incorrect package name in README
  feat(storybook-addon-theme): update icon to circle
  feat(storybook-addon-theme): ability to set theme per-story
  util: polymorphic primatives (#1690)
  build: publish
  fix(select): dropdown zindex added (#1691)
  Update @pluralsight/ps-design-system-storybook-addon-theme package.json for addon-catalog
  Update @pluralsight/ps-design-system-storybook-addon-center package.json for addon-catalog
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.

None yet

2 participants