Feature/style fixes - Githubissues

nerik commented 1 year ago

Screenshot 2023-05-24 at 11 45 15

Fixes:

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
food-system-digital-twin	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 25, 2023 0:59am

faustoperez commented 1 year ago

Looking great Erik! ⚡️

I have a few more comments:

The link in the logo should be "The Plotline" not just "Plotline"
I like the bigger icons and fonts on the sidebar. We need a bit more space around the county name on the dark header, maybe give the h2 a bit more air, something like margin: 8px 0 16px 0;
Make the border-radius on the sidebar a bit smaller, around 8px instead of 16px.
We need a hover state for the Consumer/Producer tabs, can you make them white on hover (with no line at the bottom)?
The tooltips on the destination counties are great, also the hover interaction on the map is not overwhelming. Can you try removing the little triangle pointer at the bottom of the tooltip and see how it looks?
Great job de-emphasizing the map. Can we de-emphasize Alaska as well, and limit the zoom levels and positioning so we cannot move the map to Europe?

Thanks!

wrynearson commented 1 year ago

This looks fantastic @nerik 😻 !

➕1 to what @faustoperez said about country/state de-emphasis. Right now, it seems like random countries (Kenya, Russia, and China to name a few) are de-emphasised more than others. Either checking these all or limiting zoom/pan at some point would be nice.

The main producer/consumer tooltip appears above the main sidebar. I don't think we want this, but @faustoperez could confirm.

earthrise-media / food-system-digital-twin-website