DEV Community

Discussion on: usePageTitle custom reactjs hook

Collapse
 
zcdunn profile image
Zack Dunn

Shouldn't title be in the useEffect dependency array so that it reruns the effect if title changes?

I agree with @worsnupd that you should probably restore the original title in the cleanup.

Collapse
 
worsnupd profile image
Daniel Worsnup

Agreed! I totally missed that. The effect should list title as a dependency.