Responsive header and theory page #3
Conversation
|
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/arrestierchristophe/teamhealthcheck/3g1zp78za |
|
|
||
| useEffect(() => { | ||
| setHidden(true); | ||
| }, [pathname]); |
There was a problem hiding this comment.
Not sure to understand what you are trying to achieve here... If you change location you display header again ?
There was a problem hiding this comment.
If you change location you hide the menu
|
|
||
| return ( | ||
| <header className={classNames({ medium, light: light && !sticky, sticky })}> | ||
| <span className="icon material-icons" onClick={toggleMenu}> |
There was a problem hiding this comment.
This mecanism is neat but in responsive mode i'm not satisfied with a full width header. It would be nice to have the burger menu display just in a "thumbnail" or a bookmark, like (ascii style) :
----
Menu )
----
There was a problem hiding this comment.
Without the bottom border ?
There was a problem hiding this comment.
Like, float left ?
| } | ||
|
|
||
| .menu { | ||
| background-color: $base_grey; |
There was a problem hiding this comment.
Can we have a less "dirt-like" color ? Maybe black on white or white on black ...
The best option i can think of right now is black on white with a "blur" filter on the rest of the page to hide it (like a modal)
No description provided.