Redesign showcase on splash

victorlin commented 2 weeks ago

preview

![image](https://github.com/nextstrain/nextstrain.org/assets/13424970/5eeb6a73-3f28-4f30-aa08-ccd10045fb80)

Description of proposed changes

Adding more information to the showcase ("featured analyses") section.

Links

849
Slack thread for design discussion

Checklist

[x] ~Check if changes affect the resource index JSON revision~
[x] Settle on design
[x] Settle on descriptions
[x] Clean up code
[x] Consider adding border color change on hover to mimic behavior of buttons 01bcb3cb5b801b5d0fa84e0eafa0ab9fed4e49c7
- The link underline on hover is a sufficient focus indicator for these cards.
[x] Local: swapping in blab for a group URL shows the blab group logo
[x] Checks pass
[x] Merge base PR #914
[x] Post-merge: check that the group logos are properly fetched (it's expected to not work on review app)

trvrb commented 1 week ago

I really like where this is headed @victorlin (and it seems like most everyone agrees with this). I just made some minor tweaks. I'd be happy for this to go live except the issues with /pathogens, etc... and code clean up.

I think it's your call for whether to:

Just merge this PR into PR #914 (branch trvrb/semantic-tiles) as this PR is set up in GitHub to do, and then clean up the PR there
Close PR #914 and swap this PR to merge into master

victorlin commented 1 week ago

@trvrb thanks for those tweaks, visually it's looking very polished. One more thing I want to try is border change on hover to mimic behavior of buttons on the same page.

Code is quite messy so I'll clean it up with the aim to keep things visually identical.

Since there are additional screenshot updates in this PR, I'll close #914 and consolidate all the screenshot changes into this PR as a single commit.

trvrb commented 1 week ago

Sounds good. Thanks @victorlin!

nextstrain / nextstrain.org