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

2025 methodology text #666

Merged
merged 14 commits into from
May 28, 2024
Merged

2025 methodology text #666

merged 14 commits into from
May 28, 2024

Conversation

lucascumsille
Copy link
Contributor

@lucascumsille lucascumsille commented May 22, 2024

Fixes part of: #643

  • Updated static copy
  • Updated content navigation bar
  • Updated homepage draft methodology announcement.

Note: about the "Announcement" component I have included two versions, the 2022 one, which I think is a bit hard to scan. The other versions is a two column layout one(not a huge fan of the grey box below the main content so I wouldn't mind getting rid of that). Let me know which one you prefer and I'll erase the one.

Screenshot 2024-05-22 at 12 11 42 Screenshot 2024-05-22 at 12 11 34 Screenshot 2024-05-22 at 12 13 09

@lucascumsille lucascumsille requested a review from zarino May 22, 2024 11:13
Copy link
Member

@zarino zarino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From a quick scroll through, this looks good to me. I’ve tweaked the homepage banner layout, and moved the methodology page table of contents into its own partial.

I think the next step here is for @struan to get this deployed to staging, for CEUK to review.

@zarino zarino mentioned this pull request May 28, 2024
@zarino zarino linked an issue May 28, 2024 that may be closed by this pull request
@struan struan force-pushed the 2025_methodology branch from 2704e66 to 49d39c8 Compare May 28, 2024 16:11
@struan struan force-pushed the 2025_methodology_text branch from 673a33f to 7710bf3 Compare May 28, 2024 16:15
@struan struan merged commit 7710bf3 into 2025_methodology May 28, 2024
5 checks passed
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.

Update data packages link
3 participants