Skip to content

Commit 3dc98c7

Browse files
authored
Merge pull request #395 from rvsia/fixDemoSelected
Fix selected menu items with queries
2 parents 09f902f + 65c96ec commit 3dc98c7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

packages/react-renderer-demo/src/app/src/components/navigation/find-connected-links.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const query = /\?.*/;
1+
export const query = /\?.*/;
22

33
const findConnectedLinks = (pathname, navSchema) => navSchema.find(({ link }) => pathname.replace(/^\//, '').replace(query, '') === link);
44

packages/react-renderer-demo/src/app/src/components/navigation/mapper.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import RouterNavLink from 'next/link';
1313
import { useRouter } from 'next/router';
1414

1515
import { navStyles } from './nav-styles';
16+
import { query } from './find-connected-links';
1617

1718
const useStyles = makeStyles(navStyles);
1819

@@ -23,7 +24,7 @@ const Item = ({ href, linkText, component }) => {
2324
return (
2425
<ListItem
2526
button
26-
selected={ href.replace('/?', '?') === router.asPath }
27+
selected={ href.replace('/?', '?') === router.asPath.replace(query, '') }
2728
key={ href || linkText }
2829
className={ classes.nested }
2930
component={ forwardRef((props, ref) => (

0 commit comments

Comments
 (0)