nextstrain / nextstrain.org

The Nextstrain website
https://nextstrain.org
GNU Affero General Public License v3.0
87 stars 49 forks source link

Groups page improvements #947

Open victorlin opened 1 week ago

victorlin commented 1 week ago

Potential tasks

Ideas considered but not pursued

victorlin commented 1 week ago

Remove tiles in favor of resource listing

Rather than removing it, I think it can be improved. With an expandable tiles container, /groups would look visually similar to /pathogens. Keeping the tiles maintains that similarity, though the behavior on click is slightly different.

An idea to integrate the two parts of the page:

  1. Make the click behavior of /groups act as a filter similar to the behavior on /pathogens.
  2. Make the resource listing titles clickable (to preserve links to group pages). This is currently in a PR: https://github.com/nextstrain/nextstrain.org/pull/928/commits/f137aea34b358a23c997e39aa732edaaaec4cd6b
  3. Make the resource listing show groups without resources (to preserve links to all groups).
joverlee521 commented 1 week ago

Make the resource listing show groups without resources (to preserve links to all groups).

Ah right, I hadn't considered groups without datasets! We should definitely preserve links to all groups even if they do not have published resources.

I think the expandable cards container makes sense for the /pathogens page because we plan to have curated cards. Will the expandable display still make sense if we display ~all groups (we're already at 30+ public groups) or would it feel like clutter?

joverlee521 commented 1 week ago

Use group logo/title in card

Related to https://github.com/nextstrain/nextstrain.org/issues/870 which also has a task to display group logos in the resource listing.

victorlin commented 1 week ago

Will the expandable display still make sense if we display ~all groups (we're already at 30+ public groups) or would it feel like clutter?

I'll make a PR so we can get a better sense of it interactively.