Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: On initial render, ensure that the input prompt for Autocomplete is not visible unless focused #357

Merged
merged 1 commit into from
Oct 31, 2022

Conversation

sri-viswanathan
Copy link
Collaborator

@sri-viswanathan sri-viswanathan commented Oct 31, 2022

Before: Prompt is visible on initial render even when the input does not have focus

Screenshot 2022-10-31 at 13 59 12

After: Prompt is no longer visible on initial render unless the input has focus

Screenshot 2022-10-31 at 14 03 24

Fixes #355

@sri-viswanathan sri-viswanathan added the bug Something isn't working label Oct 31, 2022
@sri-viswanathan sri-viswanathan added this to the 0.6 milestone Oct 31, 2022
@sri-viswanathan sri-viswanathan self-assigned this Oct 31, 2022
@daniele-zurico daniele-zurico merged commit fc3de54 into release/0.6 Oct 31, 2022
@daniele-zurico daniele-zurico deleted the bug/355-autocomplete branch October 31, 2022 14:23
daniele-zurico added a commit that referenced this pull request Jan 18, 2023
* feat: add value to button

* feat:select-containerId (#350)

* fix: On initial render, ensure that the input prompt for Autocomplete is not visible unless focused (#357)

* Feature/upgrade deps (#359)

* feat: upgrade deps

* feat: downgrade microbundle

* Autocomplete: Ensure the value of controlled input is never undefined (#358)

* feat: upgrade microbundle (#360)

* fix: Accessibility improvements (#361)

* Feature/autocomplete (#364)

* feat: add dynamic id to the list

* feat: add dynamic id to the list

* feat: change on the id

* feat: remove jss dependecy (#365)

* Make onChange prop optional for FormGroup component (#368)

* feat: use sass library instead of node-sass

* feat: onChange type now optional

* Separate radio and checkbox groups (#380)

* feat: remove <FormRadio> storybook (#369)

* feat: Deprecate <FormRadio> (#370)

* feat: remove <FormGroup /> and related stories

* Task create form radio (#371)

* feat: Deprecate <FormRadio>

* feat: add <FormRadio /> component

* feat: add <FormRadioCheckboxBase> component

* feat: update dev script

* feat: move <FormGroup /> to <FormRadio />

* feat: rename test FormGroup to RadioGroup

* feat: add storybook documentation

* feat: Rename StyledRadios to Styled

* feat: Fix unstyled stories

* Task create checkbox group (#373)

* feat: create <CheckboxGroup /> component

* feat: update test

* feat: fix test. Was looking for wrong element

* feat: update storybook documentation (#378)

* feat: remove "unstyled" dir from radioGroup story

* feat: update storybook doc for single checkbox (#381)

* Correct single checkbox stories (#382)

* feat: correct <FormCheckbox /> Stories

* feat: correct <FormCheckbox /> Stories

* feat: add class to fix styling (#383)

* fix: add correct components to example src (#385)

* feat: move elements outside of label (#387)

* Conditionally render aria-label (#386)

* feat: conditionally render aria-label

* feat: unit test failed eslint checks

* Feature/upgrade storybook (#392)

* feat: update storybook

* feat: update storybook

* Feature/tab index (#389)

* fix: tabIndex prop for Button CopyToClipboard Details and FormDate

* fix: tabIndex type to string

* fix: tabIndex back to number

* fix: tabIndex and missing tests

* fix: updated liveEdit files

Co-authored-by: Daniele Zurico <daniele.zurico@gmail.com>

* fix: allow closing of the options list with the Escape key instead of onBlur (#400)

* Feat/update readme (#391)

* Update CHANGELOG.md

Co-authored-by: SergioBandera <113599747+SergioBanderaGarcia@users.noreply.github.com>
Co-authored-by: Sri Viswanathan <sri@protodiv.com>
Co-authored-by: cg-julian-taylor <107050226+cg-julian-taylor@users.noreply.github.com>
Co-authored-by: Szilvi Horvath <szilvi.horvath@capgemini.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants