Skip to content

Commit 1a4a46d

Browse files
authored
respect current path change for toolpicker (github#27288)
* respect current path change for toolpicker * remove debugging * refactor imports
1 parent 44d0691 commit 1a4a46d

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

components/article/ToolPicker.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { useEffect, useState } from 'react'
2+
import { useRouter } from 'next/router'
23
import Cookies from 'js-cookie'
34
import { UnderlineNav } from '@primer/react'
45
import { sendEvent, EventType } from 'components/lib/events'
@@ -75,6 +76,7 @@ type Props = {
7576
variant?: 'subnav' | 'tabnav' | 'underlinenav'
7677
}
7778
export const ToolPicker = ({ variant = 'subnav' }: Props) => {
79+
const { asPath } = useRouter()
7880
const { defaultTool, detectedTools } = useArticleContext()
7981
const [currentTool, setCurrentTool] = useState(getDefaultTool(defaultTool, detectedTools))
8082

@@ -100,7 +102,7 @@ export const ToolPicker = ({ variant = 'subnav' }: Props) => {
100102
preserveAnchorNodePosition(document, () => {
101103
showToolSpecificContent(currentTool)
102104
})
103-
}, [currentTool])
105+
}, [currentTool, asPath])
104106

105107
function onClickTool(tool: string) {
106108
setCurrentTool(tool)

0 commit comments

Comments
 (0)