Hotfix/dropdown-menu-nav: Add pharos-elevation-level-3 token to dropdown-menu-nav #669
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
* feat: bump typscript, add main exports * feat: use yarn registry * feat: export style paths * feat: comment mixins to test * feat: export sass files * feat: export react components * feat: expose utils styles * feat: sort out exports and file extensions * feat: remove dupe export * feat: use pattern to export styles * feat: explicitly export pharos.scss * chore: add changeset
* fix: tab event * fix: update selected tab via parent * fix(tabs): update property and attribute selectors * fix(tabs): default to selected, then selected-tab, then 0 * fix(tabs): selected-tab already defaults to 0 * fix(tabs): select initial tab panel * fix(tabs): only trigger event if tab not selected * fix(tabs): update test * fix(tabs): add pharos-tabs-tab-selected event * fix(tabs): fire tab selected in updated) * chore: add changeset * Update packages/pharos/src/components/tabs/pharos-tabs.ts Co-authored-by: Dane Hillard <[email protected]> Co-authored-by: Mike Iden <[email protected]> Co-authored-by: Dane Hillard <[email protected]>
* fix(storybook): fix updateable toast stories * fix(toast): update to use internal scoping * chore: add changeset * docs: fix casing of function name in comment * docs: update code comment to reflect scoped registry behavior
* fix(pharos-site): revert pharos-site changes for typescript * fix(pharos-site): leave exports in place * fix(pharos-site): add end line
* feat(footer): remove deprecated Google Translate widget * chore: add changeset
* develop: chore: version packages (#611) Popover: Border radius styling enforcement (#610) Sheet: Component Contribution (#603) docs: add @sirrah-tam as a contributor (#607) Tokens: improve glacier-blue-40 AAA color contrast (#606) chore: bump dependencies and update resolutions (#605) Dropdown: add line rules for light variants (#599) chore: version packages (#601) feat(icon): add compare and side-panel icons (#596) chore(sidenav): fix flaky test (#600) chore: version packages (#595) Popover: new element contribution (#588)
* fix: remove `paint` value from contain properties Of all the `contain` property values, `paint` is the most noticeably problematic when overused because it can bite people by cropping off the content they intended to render within a component that has `paint` containment. For components that rely on composition at the consumer, this has caused issues more than a few times. * chore: add changeset
* feat(global): update onBackground attribute to isOnBackground * feat(global): update tokens import for pharos-site/.../color * feat(global): revert pharos-site onBackground changes * feat(global): remove .js on token import * feat(global): update on-background to is-on-background * feat(global): remove is-on-background from pharos-site * feat(global): change onBackground to isOnBackground * fix(global): change is-on-background to on-background in tokens
* refactor(sidenav): remove sidenav button * refactor(sidenav-button): remove sus comma * refactor(sidenav-button): remove test for sidenav button * fix(sidenav): remove automatic sliding behavior * feat(sidenav): render button conditionally and add unit tests * refactor(sidenav-button): add changeset * Remove console log statement Co-authored-by: Dane Hillard <[email protected]> * fix(sidenav): fix storybook --------- Co-authored-by: Evan Shoup <[email protected]> Co-authored-by: Jialin He <[email protected]> Co-authored-by: Jialin He <[email protected]> Co-authored-by: Evan Shoup <[email protected]> Co-authored-by: Dane Hillard <[email protected]>
* develop: A11y revamp: Pharos buttons (non-breaking change) (#628) Radio, Checkbox: Fix group label (#652) Add elevation tokens and documentation (#643) fix(sidenav-link): external link opens in new tab (#645) Upgrade to TypeScript 5 (#644) feat(cli): add newly created components created using pharos-cli to initComponents files (#630) chore: version packages (#640) Coach Mark: Fix react component positioning (#638) Coach Mark: Documentation fixes (#639) chore(deps): bump @babel/traverse from 7.20.0 to 7.23.2 (#637) chore: version packages (#636) Icon: Add Panorama icon (#631) chore: version packages (#629) Loading spinner: add small and on background variant (#627) chore: version packages (#626) Sheet: allow expansion with attribute (#625) fix(button): remove fill on subtle disabled button on background (#618) chore(deps-dev): bump postcss from 8.4.25 to 8.4.31 (#624) chore: version packages (#623) Sheet: Add more close options and transition timing function (#620)
* feat(button): add ability to pass down ARIA attributes * feat(button): add reference to new button types Including the ExpandedState type reference for other components that consume the button. These threw errors when initially compiling so there may be other such commponents that eventually need a similar update. * feat(button): attempt at updating storybook example * feat(button): remove test code for popupstate * feat(button): add aria-haspopup * chore(changeset): add changeset * feat(button): update label attr to a11y-label * feat(button): replace property for ButtonVariant * feat(button): allow backwards compatibility Gives warning if using deprecated attributes, updating aria-pressed * feat(button): update storybook aria-pressed * feat(button): remove fallback from major release * feat(button): add a11y attributes typing * test(button): add tests for new aria attributes * feat(button): add ability to pass down ARIA attributes * feat(button): add reference to new button types Including the ExpandedState type reference for other components that consume the button. These threw errors when initially compiling so there may be other such commponents that eventually need a similar update. * feat(button): attempt at updating storybook example * feat(button): remove test code for popupstate * feat(button): add aria-haspopup * chore(changeset): add changeset * feat(button): update label attr to a11y-label * feat(button): replace property for ButtonVariant * feat(button): allow backwards compatibility Gives warning if using deprecated attributes, updating aria-pressed * feat(button): update storybook aria-pressed * feat(button): remove fallback from major release * feat(button): add a11y attributes typing * test(button): add tests for new aria attributes * fix(a11y attributes): update AriaHiddenState name * fix(button): remove ts ignore * feat(button): remove backwards compat * feat(button): add aria-disabled back * test(button): add a11y-label to new components * fix: remove sidenav button from bad merge * test(button): fix aria-pressed in toggle buttons * Update packages/pharos/src/components/sidenav/PharosSidenav.react.stories.jsx Co-authored-by: Dane Hillard <[email protected]> * Update packages/pharos/src/components/toast/pharos-toast-button.ts Co-authored-by: Dane Hillard <[email protected]> * fix(btn): remove aria-description --------- Co-authored-by: Dane Hillard <[email protected]>
* fix(loading-spinner, sidenav, docs): use is-on-background correctly * chore: add changeset * fix: address linting issues
Bumps [msgpackr](https://github.com/kriszyp/msgpackr) from 1.7.2 to 1.10.1. - [Release notes](https://github.com/kriszyp/msgpackr/releases) - [Commits](https://github.com/kriszyp/msgpackr/commits/v1.10.1) --- updated-dependencies: - dependency-name: msgpackr dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update icon.docs.mdx added empty space to fix run on from "an" "aria" * Update packages/pharos-site/static/guidelines/icon.docs.mdx --------- Co-authored-by: Dane Hillard <[email protected]>
🦋 Changeset detectedLatest commit: efbf45f The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
size-limit report 📦
|
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.
This change: (check at least one)
Is this a breaking change? (check one)
Is the: (complete all)
What does this change address?
Devs have to add elevation values to
pharos-dropdown-menu-navafter adding the component. We want to cut that extra step out so that the elevation is built into the component.Closes #661
How does this change work?
We added the token
pharos-elevation-level-3to the styling of the component