-
Notifications
You must be signed in to change notification settings - Fork 15
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(site): use Pharos v14 properties #703
Conversation
- Add `open` to Sidenav - Update `onBackground` to `isOnBackground` in all instances
🦋 Changeset detectedLatest commit: c7c8fbe The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
@@ -55,7 +55,7 @@ const Sidenav: FC = () => { | |||
}; | |||
|
|||
const content = ( | |||
<PharosSidenav mainContentId="skip-link" className={sidenav}> | |||
<PharosSidenav mainContentId="skip-link" open={true} className={sidenav}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note this will only partially fix this - we will be missing the expected mobile behavior. I think this is the right thing to do for now though, but will need a followup story probably too
size-limit report 📦
|
* develop: (77 commits) chore(deps): update GitHub Actions and Node versions (#704) fix(site): use Pharos v14 properties (#703) fix(storybook): update Babel configuration (#702) fix(infra): use Node 18 in .nvmrc (#701) fix(infra): update dependencies fix(infra): allow mutable Yarn lockfile during release for workspaces versions Version Packages (#700) Release/14.0.0 (#697) chore(deps): upgrade to Yarn 4 (#694) chore(deps): bump es5-ext from 0.10.62 to 0.10.64 (#692) fix(icon): add media query style for WHCM (#689) chore(deps): bump ip from 1.1.8 to 1.1.9 (#687) chore: version packages (#685) Icon: Add "add to folder" icon (#678) Fix the all-contributors badge (#680) Update link, dropdown-menu-nav, and popover to use a11y-label (#675) DropdownMenuNav: Add pharos-elevation-level-3 token (#670) chore(deps-dev): bump vite from 4.3.9 to 4.5.2 (#672) chore: version packages (#667) Update deprecated a11y attributes in button component (#666) ...
This change: (check at least one)
Is this a breaking change? (check one)
Is the: (complete all)
What does this change address?
Sidenav not open, footer and other component colors incorrect throughout the Pharos site
How does this change work?
open
to SidenavonBackground
toisOnBackground
in all instances