Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Nextclade section on splash #902

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Jun 6, 2024

preview

image

Description of proposed changes

Now that sections aren't required to have preview cards, a section for Nextclade without any cards can visually fit in.

Related issue(s)

Open questions

  1. Where should the new Nextclade section be placed? I added it to the end but it should probably go elsewhere. done
  2. Does the description/button text look ok?

Checklist

@victorlin victorlin self-assigned this Jun 6, 2024
@nextstrain-bot nextstrain-bot temporarily deployed to nextstrain-s-victorlin--n9zj3t June 6, 2024 18:01 Inactive
@victorlin victorlin mentioned this pull request Jun 6, 2024
5 tasks
@victorlin victorlin requested a review from a team June 6, 2024 18:09
@victorlin victorlin mentioned this pull request Jun 6, 2024
4 tasks
@victorlin victorlin marked this pull request as ready for review June 6, 2024 18:10
@victorlin victorlin force-pushed the victorlin/use-showcase-on-splash branch 4 times, most recently from 7b99c6a to eefb79e Compare June 10, 2024 20:32
@victorlin victorlin force-pushed the victorlin/add-nextclade-section-on-splash branch from 606598d to c423a51 Compare June 10, 2024 20:39
@victorlin victorlin temporarily deployed to nextstrain-s-victorlin--n9zj3t June 10, 2024 20:39 Inactive
@victorlin victorlin force-pushed the victorlin/use-showcase-on-splash branch 2 times, most recently from 66b395f to e6fa8aa Compare June 10, 2024 23:26
@victorlin victorlin force-pushed the victorlin/add-nextclade-section-on-splash branch from c423a51 to 9ae59a6 Compare June 10, 2024 23:51
@victorlin victorlin temporarily deployed to nextstrain-s-victorlin--n9zj3t June 10, 2024 23:51 Inactive
@victorlin victorlin force-pushed the victorlin/use-showcase-on-splash branch from e6fa8aa to a19026e Compare June 11, 2024 00:00
@victorlin victorlin force-pushed the victorlin/add-nextclade-section-on-splash branch from 9ae59a6 to 9dd6249 Compare June 11, 2024 00:00
@victorlin victorlin temporarily deployed to nextstrain-s-victorlin--n9zj3t June 11, 2024 00:00 Inactive
Now that sections aren't required to have preview cards, a section for
Nextclade without any cards can visually fit in.

Place it as the 2nd section both due to its importance and because the
abstract length is similar to SARS-CoV-2's, rendering nicely in the
2-column format.
@victorlin victorlin force-pushed the victorlin/add-nextclade-section-on-splash branch from 9dd6249 to 3878dde Compare June 12, 2024 21:52
@victorlin victorlin temporarily deployed to nextstrain-s-victorlin--n9zj3t June 12, 2024 21:52 Inactive
Base automatically changed from victorlin/use-showcase-on-splash to master June 13, 2024 17:46
@victorlin victorlin merged commit e70f530 into master Jun 13, 2024
7 checks passed
@victorlin victorlin deleted the victorlin/add-nextclade-section-on-splash branch June 13, 2024 17:47
This was referenced Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants