nit: is it more readable if we use `advancedUl.hidden = !show` instead of having a if-else? (same as `advancedMenu.hidden`)