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(TableToolbarSearch): ensure search cannot be expanded when disabled #7612

Merged

Conversation

tw15egan
Copy link
Collaborator

Closes #7522

Disabled styles will now prevent the search from expanding if disabled is passed to TableToolbarSearch

Changelog

New

  • Disabled styles for non-expanded disabled Search button
  • Logic to prevent expansion if disabled is passed to TableToolbarSearch
  • Removes TableToolbarSearch from the tab order if it is disabled

Testing / Reviewing

Pass disabled to TableToolbarSearch in the DataTable-filtering.js file and make sure the search is not able to interact with

@netlify
Copy link

netlify bot commented Jan 20, 2021

Deploy preview for carbon-elements ready!

Built with commit c7218a6

https://deploy-preview-7612--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Jan 20, 2021

Deploy preview for carbon-components-react ready!

Built with commit c7218a6

https://deploy-preview-7612--carbon-components-react.netlify.app

@tw15egan
Copy link
Collaborator Author

@emyarod let me know if there are any other issues with this 👍

@tw15egan
Copy link
Collaborator Author

@emyarod updated

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

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

should we do the same for the transition overrides as well? otherwise looks good to me

@tw15egan tw15egan force-pushed the table-toolbar-search-disabled branch from 6d3d26a to a4c1a74 Compare January 29, 2021 17:46
@tw15egan tw15egan force-pushed the table-toolbar-search-disabled branch from a4c1a74 to c7218a6 Compare January 29, 2021 17:49
@kodiakhq kodiakhq bot merged commit 6eaceac into carbon-design-system:master Jan 29, 2021
@tw15egan tw15egan deleted the table-toolbar-search-disabled branch March 3, 2022 16:12
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.

TableToolbarSearch isDisabled doesn't disable the expand
3 participants