closes #414 - Githubissues

vercel[bot] commented 10 months ago

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

1 Ignored Deployment

cloudflare-pages[bot] commented 10 months ago

Deploying with Cloudflare Pages

Latest commit:	`9cc2550`
Status:	✅ Deploy successful!
Preview URL:	https://abfa7a33.correlaid.pages.dev
Branch Preview URL:	https://414-icons-navigation.correlaid.pages.dev

jstet commented 10 months ago

waiting for confirmation via slack

friep commented 10 months ago

Thanks! I like the elements and think that they could be useful on other pages.

Looking at them on my big ass monitor screen, i have two concerns:

font: i don't think the font is well readable from an accessibility perspective when it is so small. i know we have been using it on canva for a bit but i feel for this, it is not so well suited (the letters "mush together" even for me).
colors: maybe we could think about different colors for the backgrounds of the icons? I think right now it implies too much of a "sequence". in [correltools](https://github.com/CorrelAid/correltools) , there the opportunity to create qualitative palettes:

scales::show_col(correltools::correlaid_pal()(5))

Screenshot 2023-05-31 at 15 37 10

or is this too many different colors? IDK.

jstet commented 10 months ago

@friep i have increased the font spacing, size etc. Does it work better now?

I intentionally used a sequence of colours so it fits to the gradient of the hero etc.

jstet commented 10 months ago

I will merge for now because i edited texts in the cms that are based on this change and this is blocking the build for other changes.