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(breadcrumb): remove firefox border override #903

Merged
merged 3 commits into from
Jun 19, 2018

Conversation

tw15egan
Copy link
Contributor

Overview

Resolves #902

Removes specific Firefox override that sets Breadcrumb border to none. This causes layout jumping.

Removed

  • Specific Firefox override that sets Breadcrumb border to none.

Copy link
Member

@alisonjoseph alisonjoseph left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @tw15egan!

@tw15egan tw15egan merged commit 07e5cdf into carbon-design-system:master Jun 19, 2018
@carbon-bot
Copy link
Contributor

🎉 This PR is included in version 9.0.12 🎉

The release is available on:

Your semantic-release bot 📦🚀

joshblack added a commit to joshblack/carbon that referenced this pull request May 2, 2019
joshblack pushed a commit to joshblack/carbon that referenced this pull request May 2, 2019
* chore: Update .babelrc, resolves  carbon-design-system#885 (carbon-design-system#886)

* chore: Update .babelrc, resolves  carbon-design-system#885

* chore: Update pacage.json, resolves  carbon-design-system#885

* chore(build): fix babel config for test (carbon-design-system#892)

* feat(DatePicker): Added minDate and maxDate props and pass them on to flatpickr (carbon-design-system#887)

* feat(DatePicker): Added minDate and maxDate props and pass them on to flatpickr

* Updated story description

* fix: Pagination component's range and page text for no items (carbon-design-system#872)

* fix: Pagination component's range and page text for no items, resolves carbon-design-system#553

* fix: Update UTs for PaginationV2

* feat(SearchFilterButton): add support for attaching event handlers (carbon-design-system#896)

* fix(DatePicker): treat Flatpickr's default export as a constructor (carbon-design-system#890)

Fixes carbon-design-system#491.

* fix: console.errors coming from using Datepicker (carbon-design-system#899)

* fix(package): update flatpickr to version 4.5.0 (carbon-design-system#897)

* fix(package): update flatpickr to version 4.5.0

Closes carbon-design-system#893

* chore(package): update lockfile

https://npm.im/greenkeeper-lockfile

* fix(table-toolbar-search): add i18n support (carbon-design-system#903)

* fix(OverflowMenu): fix <ClickOutside> not covering floating menu (carbon-design-system#906)

Fixes carbon-design-system#628.

* fix(icon-skeleton): Add missing export for IconSkeleton component (carbon-design-system#917)

* fix(Search): make search icon description translatable (carbon-design-system#927)

Fixes carbon-design-system#923.

* fix(Tooltip): prevent aria-owns attribute being set unless the target really exists (carbon-design-system#926)

Fixes carbon-design-system#913.

* feat(slider): Add light prop to slider
joshblack pushed a commit to joshblack/carbon that referenced this pull request May 2, 2019
…Menu's open state (carbon-design-system#889)

* feat(OverflowMenu): added onClose to props to notify menu is closed

* feat(OverflowMenu): moved onClose handler to componentDidUpdate

* feat(OverflowMenu): moved onClose handler to render() where state.open is surely changed

* feat(OverflowMenu): moved onClose handler to componentDidUpdate

* feat(OverflowMenu): add onOpen to notify caller menu's open state

* feat(OverflowMenu): moved onClose & onOpen handler to componentDidUpdate

* feat(OverflowMenu): added onClose to props to notify menu is closed

* chore(build): fix babel config for test (carbon-design-system#892)

* feat(OverflowMenu): moved onClose handler to componentDidUpdate

* feat(OverflowMenu): moved onClose handler to render() where state.open is surely changed

* feat(OverflowMenu): moved onClose handler to componentDidUpdate

* feat(DatePicker): Added minDate and maxDate props and pass them on to flatpickr (carbon-design-system#887)

* feat(DatePicker): Added minDate and maxDate props and pass them on to flatpickr

* Updated story description

* fix: Pagination component's range and page text for no items (carbon-design-system#872)

* fix: Pagination component's range and page text for no items, resolves carbon-design-system#553

* fix: Update UTs for PaginationV2

* feat(SearchFilterButton): add support for attaching event handlers (carbon-design-system#896)

* fix(DatePicker): treat Flatpickr's default export as a constructor (carbon-design-system#890)

Fixes carbon-design-system#491.

* fix: console.errors coming from using Datepicker (carbon-design-system#899)

* fix(package): update flatpickr to version 4.5.0 (carbon-design-system#897)

* fix(package): update flatpickr to version 4.5.0

Closes carbon-design-system#893

* chore(package): update lockfile

https://npm.im/greenkeeper-lockfile

* fix(table-toolbar-search): add i18n support (carbon-design-system#903)

* fix(OverflowMenu): fix <ClickOutside> not covering floating menu (carbon-design-system#906)

Fixes carbon-design-system#628.

* feat(OverflowMenu): add onOpen to notify caller menu's open state

* feat(OverflowMenu): moved onClose & onOpen handler to componentDidUpdate

* feat(OverflowMenu): moved onOpen call hook to_handlePlace

* feat(OverflowMenu): refine from Josh and Akira's reviews

* feat(OverflowMenu): refine from Josh and Akira's reviews

* feat(OverflowMenu): refine from Josh and Akira's reviews
@tw15egan tw15egan deleted the breadcrumb-fix branch April 28, 2021 18:14
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.

4 participants