[web] [fix] [ENG-986] replace useLayoutEffect with useEffect

Summary:
useLayoutEffect isn't needed here

https://linear.app/comm/issue/ENG-986/regression-uselayouteffect-warning-in-keyserver-console

Test Plan: open and close the menu button in header

Reviewers: atul, def-au1t, palys-swm

Reviewed By: def-au1t, palys-swm

Subscribers: ashoat, palys-swm, Adrian, karol-bisztyga, yayabosh

Differential Revision: https://phabricator.ashoat.com/D3678