Skip to content

Conversation

@lucascumsille
Copy link
Contributor

@lucascumsille lucascumsille commented Jan 20, 2026

  • Added /themes page to list all existing themes
  • Also list all the themes in the /sections page
  • For the theme single pages infographics are displayed.
Screenshot 2026-01-20 at 13 55 02 Screenshot 2026-01-20 at 13 55 24 Screenshot 2026-01-20 at 13 55 41
Screen.Recording.2026-01-20.at.13.55.48.mov
Screen.Recording.2026-01-20.at.13.56.09.mov

@lucascumsille lucascumsille requested a review from zarino January 20, 2026 14:03
@lucascumsille
Copy link
Contributor Author

@zarino I'll move the thematics to the social-graphic.py

@lucascumsille lucascumsille marked this pull request as draft January 21, 2026 06:48
- Added `/themes` page to list all existing themes
- Also list all the themes in the `/sections` page
- For the theme single pages infographics are displayed.
@lucascumsille lucascumsille marked this pull request as ready for review January 21, 2026 09:37
@lucascumsille
Copy link
Contributor Author

@zarino I just rebased with master and moved the thematic infographics to the social-graphics.py file.

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.

This largely looks good to me! Only one small thing I spotted.

Guess we should keep this PR in review until the Cost Saving infographic is done, then we can launch both together?

In the meantime, you could always come along to our next catchup call with CEUK and show them what we’re planning, see whether they have any comments!

"adaptation": {
"pdf": {
"src_pdf": "scoring/img/social-graphics-2025/adaptation.pdf",
"src_jpg": "scoring/img/social-graphics-2025/adaptation.png",
Copy link
Member

Choose a reason for hiding this comment

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

A minor thing, but ideally these src_jpg files really should be jpegs and not pngs, because we mark them up as having a image/jpeg mime type in the opengraph tags. (OpenGraph parsers probably then go on to ignore the tags, but 🤷.) It looks like we’ve used a mixture of jpg and png files for previous sections/nations.

Copy link
Member

Choose a reason for hiding this comment

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

If we want to make some of them jpgs and some of them pngs (eg: for filesize optimisation) then we could potentially change the template code to infer the right mime type from the file extension 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I'll delete the PNGs and just have jpgs

@lucascumsille lucascumsille marked this pull request as draft January 27, 2026 14:39
@lucascumsille
Copy link
Contributor Author

We are awaiting for some copy related from CEUK, before merging.

Regarding the opengraph format. I'll handle that on another PR where I'll replace them with png and stop using jpg.

- Replaced all JPG main images for graphics with PNG
- Replaced button label from JPG to PNG
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.

3 participants