-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Commit 0bfd30d
authored
[Navigation] Consolidate se23 logic and styles (#10213)
### WHY are these changes introduced?
Resolves #9952.
### WHAT is this pull request doing?
Consolidates se23 beta styles and logic for `Navigation`.
### How to 🎩
[Storybook](https://5d559397bae39100201eedc1-lecmsuzwza.chromatic.com/?path=/story/all-components-navigation--default)
[Prod
Storybook](https://storybook.polaris.shopify.com/?path=/story/all-components-navigation--default&globals=polarisSummerEditions2023:true)
🖥 [Local development
instructions](https://github.com/Shopify/polaris/blob/main/README.md#local-development)
🗒 [General tophatting
guidelines](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md)
📄 [Changelog
guidelines](https://github.com/Shopify/polaris/blob/main/.github/CONTRIBUTING.md#changelog)
### 🎩 checklist
- [x] Tested on
[mobile](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting.md#cross-browser-testing)
- [x] Tested on [multiple
browsers](https://help.shopify.com/en/manual/shopify-admin/supported-browsers)
- [ ] Tested for
[accessibility](https://github.com/Shopify/polaris/blob/main/documentation/Accessibility%20testing.md)
- [ ] Updated the component's `README.md` with documentation changes
- [ ] [Tophatted
documentation](https://github.com/Shopify/polaris/blob/main/documentation/Tophatting%20documentation.md)
changes in the style guide1 parent 5101d22 commit 0bfd30dCopy full SHA for 0bfd30d
File tree
Expand file treeCollapse file tree
4 files changed
+304
-557
lines changedOpen diff view settings
Filter options
- polaris-react/src/components/Navigation
- components/Item
- components/SecondaryNavigation
Expand file treeCollapse file tree
4 files changed
+304
-557
lines changedOpen diff view settings
0 commit comments