Skip to content

Commit 6c8acf9

Browse files
author
Eric Olkowski
committed
Use style object for classes and update hook deps
1 parent ffad95b commit 6c8acf9

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

src/components/PageToggle.tsx

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import type React from 'react'
2+
import styles from '@patternfly/react-styles/css/components/Page/page'
23
import { PageToggleButton } from '@patternfly/react-core'
34
import BarsIcon from '@patternfly/react-icons/dist/esm/icons/bars-icon'
45
import { useStore } from '@nanostores/react'
@@ -21,14 +22,14 @@ export const PageToggle: React.FunctionComponent = () => {
2122
return
2223
}
2324

24-
if (!sideBarIsland.classList.contains('pf-v6-c-page__sidebar')) {
25+
if (!sideBarIsland.classList.contains(styles.pageSidebar)) {
2526
sideBarIsland.classList.add(
26-
'pf-v6-c-page__sidebar',
27-
$isNavOpen ? 'pf-m-expanded' : 'pf-m-collapsed',
27+
styles.pageSidebar,
28+
$isNavOpen ? styles.modifiers.expanded : styles.modifiers.collapsed,
2829
)
2930
} else {
30-
sideBarIsland.classList.toggle('pf-m-expanded')
31-
sideBarIsland.classList.toggle('pf-m-collapsed')
31+
sideBarIsland.classList.toggle(styles.modifiers.expanded)
32+
sideBarIsland.classList.toggle(styles.modifiers.collapsed)
3233
}
3334
sideBarIsland.setAttribute('aria-hidden', `${!$isNavOpen}`)
3435
}
@@ -39,7 +40,7 @@ export const PageToggle: React.FunctionComponent = () => {
3940

4041
useEffect(() => {
4142
applySidebarStylesToIsland()
42-
}, [$isNavOpen])
43+
}, [$isNavOpen, applySidebarStylesToIsland])
4344

4445
return (
4546
<PageToggleButton

0 commit comments

Comments
 (0)