Skip to content

Conversation

@prsdthkr
Copy link
Member

@prsdthkr prsdthkr commented Sep 14, 2020

Description

In this change, we fix some bugs in Combobox Input.

  • remove unconditional popover closing that caused scroll jumps in storybook docs
  • updated list styles to match the latest spec in fd-styles
  • some refactoring to fix a prop type error

Before

image

image

After

image

fixes #1205

@prsdthkr prsdthkr self-assigned this Sep 14, 2020
@prsdthkr prsdthkr added the bug Something isn't working / Issues in the code label Sep 14, 2020
@netlify
Copy link

netlify bot commented Sep 14, 2020

Deploy preview for fundamental-react ready!

Built with commit bf97924

https://deploy-preview-1206--fundamental-react.netlify.app

@prsdthkr prsdthkr marked this pull request as ready for review September 14, 2020 23:50
Copy link
Contributor

@jbadan jbadan left a comment

Choose a reason for hiding this comment

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

Besides the issue you already opened about validation states+ combobox, this looks good to me! 🚢

@prsdthkr prsdthkr merged commit 493e867 into master Sep 15, 2020
@prsdthkr prsdthkr deleted the fix/combobox-bugs branch September 15, 2020 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working / Issues in the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clicking anywhere on the Combobox page causes auto scrolling

3 participants