fix(filter-panel-accordion-item): use outline and transparent border to prevent hover + focus overlap #378
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…to prevent hover + focus overlap
|
Deploy preview for ibm-security ready! Built with commit ff619f5 |
|
@SimonFinney Would you have a chance to review this? Small bug fix. Would be nice to get @carbon-design-system/ibm-security-design to review as well. |
SimonFinney
reviewed
Feb 25, 2020
src/components/FilterPanel/FilterPanelAccordionItem/_mixins.scss
Outdated
Show resolved
Hide resolved
|
Thanks @SimonFinney! Just applied the |
SimonFinney
pushed a commit
that referenced
this pull request
Feb 26, 2020
## [1.17.2-prerelease.6](v1.17.2-prerelease.5...v1.17.2-prerelease.6) (2020-02-26) ### Bug Fixes * **filter-panel-accordion-item:** use outline and transparent border to prevent hover + focus overlap ([#378](#378)) ([5edb4b1](5edb4b1))
|
🎉 This PR is included in version 1.17.2-prerelease.6 🎉 The release is available on: Your semantic-release bot 📦🚀 |
SimonFinney
pushed a commit
that referenced
this pull request
Mar 3, 2020
# [1.18.0](v1.17.1...v1.18.0) (2020-03-03) ### Bug Fixes * **button:** ensure loading icon does not make button taller ([#395](#395)) ([32f8992](32f8992)) * **error-page:** update error page tests and remove snapshots ([#358](#358)) ([ebe7b47](ebe7b47)) * **external-link:** add a11y and attributes tests, remove snapshots ([#365](#365)) ([fec99c1](fec99c1)) * **filter-panel:** accordion item themed gradient color ([#394](#394)) ([059d5cf](059d5cf)) * **filter-panel-accordion-item:** use outline and transparent border to prevent hover + focus overlap ([#378](#378)) ([5edb4b1](5edb4b1)) * **ica:** update tests for ica and skeleton variation ([#379](#379)) ([f9969a3](f9969a3)) * **interactive-tag:** update tests and remove old snapshots ([#382](#382)) ([f84fd45](f84fd45)) * **navitem:** prevent flex shrink of external link icon, update story demo ([#409](#409)) ([78e0579](78e0579)) * **type-layout:** flatten styles and do not clone cells ([#408](#408)) ([55fde34](55fde34)) * **type-layout:** update tests and remove old snapshots ([#383](#383)) ([0b99110](0b99110)) * **UNSTABLE__pagination:** add a11y and unit tests ([#327](#327)) ([9490e4f](9490e4f)) ### Features * **card-skeleton:** allow custom classname and extra props via spread attr ([#405](#405)) ([480a9ed](480a9ed))
|
🎉 This PR is included in version 1.18.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
SimonFinney
pushed a commit
that referenced
this pull request
Mar 12, 2020
# [2.4.0](v2.3.0...v2.4.0) (2020-03-12) ### Bug Fixes * **button:** ensure loading icon does not make button taller ([#395](#395)) ([32f8992](32f8992)) * **button:** update testing structure, props docs, and add a11y test ([#362](#362)) ([0ee20bf](0ee20bf)) * **checkbox-skeleton:** correctly export skeleton component ([#422](#422)) ([911e8f8](911e8f8)) * **error-page:** update error page tests and remove snapshots ([#358](#358)) ([ebe7b47](ebe7b47)) * **external-link:** add a11y and attributes tests, remove snapshots ([#365](#365)) ([fec99c1](fec99c1)) * **filter-panel:** accordion item themed gradient color ([#394](#394)) ([059d5cf](059d5cf)) * **filter-panel:** add a11y tests to subcomponents, use test keyword ([#375](#375)) ([a04cb8f](a04cb8f)) * **filter-panel-accordion-item:** use outline and transparent border to prevent hover + focus overlap ([#378](#378)) ([5edb4b1](5edb4b1)) * **ica:** update tests for ica and skeleton variation ([#379](#379)) ([f9969a3](f9969a3)) * **interactive-tag:** update tests and remove old snapshots ([#382](#382)) ([f84fd45](f84fd45)) * **navitem:** prevent flex shrink of external link icon, update story demo ([#409](#409)) ([78e0579](78e0579)) * **string-formatter:** update tests ([#413](#413)) ([e5edb22](e5edb22)) * **summary-card:** add a11y and unit tests ([#337](#337)) ([78ceb90](78ceb90)) * **summary-card-action:** remove overflow hidden so tooltip can be shown ([#420](#420)) ([fb72d41](fb72d41)) * **trending-card:** reformat tests and remove snapshots ([#346](#346)) ([423f352](423f352)) * **type-layout:** flatten styles and do not clone cells ([#408](#408)) ([55fde34](55fde34)) * **type-layout:** update tests and remove old snapshots ([#383](#383)) ([0b99110](0b99110)) * **UNSTABLE__pagination:** add a11y and unit tests ([#327](#327)) ([9490e4f](9490e4f)) ### Features * **card-skeleton:** allow custom classname and extra props via spread attr ([#405](#405)) ([480a9ed](480a9ed))
|
🎉 This PR is included in version 2.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Affected issues
Proposed changes
FilterPanelAccordionItem's hover state from overlapping the focus outline of an adjacentFilterPanelAccordionItem