Add a header to the app

acteng / atip

Active Travel Infrastructure Platform

https://acteng.github.io/atip/

Apache License 2.0

22 stars 4 forks source link

Add a header to the app #518

Closed Pete-Y-CS closed 3 weeks ago

Pete-Y-CS commented 3 weeks ago

hmm will investigate test failures

Pete-Y-CS commented 3 weeks ago

tests fixed 🙌🏻

dabreegster commented 3 weeks ago

Cool to see this! In Figma, the banner is just covering the left sidebar, not the full screen: I'll work on making that change

dabreegster commented 3 weeks ago

I matched Figma and simplified the code. Trying to share the same CSS for all 3 pages doesn't quite work. There's more changes in Figma like the service banner: and I have some questions about whether to display all of this in every mode of SS (as you activate a drawing tool, for instance), but we can wait to get designs for that.

dabreegster commented 3 weeks ago

Also a bit odd to cover up part of the banner with the collapse tool, but I'm guessing that control will be redesigned soon anyway