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.

Sign up for GitHub

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

Jump to bottom

Port /sars-cov-2/forecasts to App Router #1125

Closed
Tracked by #1052
genehack opened this issue Feb 26, 2025 · 0 comments · Fixed by #1126
Closed
Tracked by #1052

Port /sars-cov-2/forecasts to App Router #1125

genehack opened this issue Feb 26, 2025 · 0 comments · Fixed by #1126
Assignees
@genehack

Comments

@genehack
Copy link
Contributor

No description provided.

@genehack genehack changed the title /sars-cov-2/forecasts Port /sars-cov-2/forecasts to App Router Feb 26, 2025
@genehack genehack mentioned this issue Feb 26, 2025
Open
32 tasks
@genehack genehack self-assigned this Feb 26, 2025
genehack added a commit that referenced this issue Feb 28, 2025
@genehack
Initial move of /sars-cov-2/forecasts page into App Router style [#1125]
17b327a 
Note: this will leave the site in a broken state; doing this to ensure
better Git tracking of file renames.
genehack added a commit that referenced this issue Feb 28, 2025
@genehack
Port /sars-cov-2/forecasts to App Router [#1125]
44668a1 
* Move styles into CSS module
* Refactor code for consistency with other App Router pages
* Switch abstract section from <FocusParagraph> to
  <FocusParagraphNarrow> for consistency with other pages
* Add Metadata export
@genehack genehack mentioned this issue Feb 28, 2025
Merged
2 tasks
genehack added a commit that referenced this issue Mar 6, 2025
@genehack
Initial move of /sars-cov-2/forecasts page into App Router style [#1125]
0ae4184 
Note: this will leave the site in a broken state; doing this to ensure
better Git tracking of file renames.
genehack added a commit that referenced this issue Mar 6, 2025
@genehack
Port /sars-cov-2/forecasts to App Router [#1125]
e17cfc1 
* Move styles into CSS module
* Refactor code for consistency with other App Router pages
* Switch abstract section from <FocusParagraph> to
  <FocusParagraphNarrow> for consistency with other pages
* Add Metadata export
genehack added a commit that referenced this issue Mar 6, 2025
@genehack
Port /sars-cov-2/forecasts to App Router [#1125]
af42511 
* Move styles into CSS module
* Refactor code for consistency with other App Router pages
* Switch abstract section from <FocusParagraph> to
  <FocusParagraphNarrow> for consistency with other pages
* Add Metadata export

NOTE: This changes the width of the "abstract" section of the page so
that it matches other pages on the site; previously, the "abstract"
section of the `/sars-cov-2/forecasts` was wider than the 640px
default used on other pages.

Having everything use a consistent layout is an obvious maintenance
win; I picked the 640px variation because that's more common across
all the existing site pages.
genehack added a commit that referenced this issue Mar 10, 2025
@genehack
Initial move of /sars-cov-2/forecasts page into App Router style [#1125]
d8e596c 
Note: this will leave the site in a broken state; doing this to ensure
better Git tracking of file renames.
genehack added a commit that referenced this issue Mar 10, 2025
@genehack
Port /sars-cov-2/forecasts to App Router [#1125]
6b91575 
* Move styles into CSS module
* Refactor code for consistency with other App Router pages
* Switch abstract section from <FocusParagraph> to
  <FocusParagraphNarrow> for consistency with other pages
* Add Metadata export

NOTE: This changes the width of the "abstract" section of the page so
that it matches other pages on the site; previously, the "abstract"
section of the `/sars-cov-2/forecasts` was wider than the 640px
default used on other pages.

Having everything use a consistent layout is an obvious maintenance
win; I picked the 640px variation because that's more common across
all the existing site pages.
genehack added a commit that referenced this issue Mar 10, 2025
@genehack
Merge pull request #1126 from nextstrain/convert-sars-forecasts-to-ap…
279f3b4 
…p-router-1125

Convert sars forecasts to app router [#1125]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@genehack genehack

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Convert sars forecasts to app router [#1125] nextstrain/nextstrain.org
1 participant
@genehack