Skip to content

Conversation

marioguzzzman
Copy link

@marioguzzzman marioguzzzman commented Aug 21, 2025

@perminder-17 @Homaid @ksen0

Following this discussion> #931 (comment)

I think the quotes create issues that go beyond consistency.

Issue identified: YAML values with quotes are not being processed correctly by the template engine.

From live website:

  • Working translations (no quotes): Main content renders correctly in es/ko/hi/zh-Hans
  • Broken translations (with quotes): Tutorial sidebar categories remain in English only

The tutorialCategoriessection is using quoted values while other working sections use unquoted values. This suggests the p5.js template engine has a parsing issue with quoted YAML strings.

Proposed solution: Remove quotes from all tutorialCategories values.

Screenshots showing the issue across different languages:

Screenshot 2025-08-19 171020 Screenshot 2025-08-19 171034 Screenshot 2025-08-19 171049

- Removes quotes from YAML values to fix template engine parsing
- Resolves sidebar translation issue where categories stayed in English
- Fixes follow-up from PR processing#931"
@ksen0 ksen0 added the Bug Something isn't working label Aug 25, 2025
@ksen0
Copy link
Member

ksen0 commented Aug 25, 2025

Hi @marioguzzzman thanks so much for raising this! Right now, I'm not able to reproduce the issue - eg I go to https://p5js.org/es/tutorials/ and I see:

image

This is the current live build, which does have a mix of quotes. (Similarly for ko/hi/zh-Hans)
Did I misunderstand something? I haven't encountered yaml parsing issues so far, but definitely happy to more testing/attention on the internationalization workflow!

@marioguzzzman
Copy link
Author

@ksen0 You are right. Now, it seems like the sidebar is displaying the languages correctly. Before, as showed in my screenshots, it was not...
I did paid attention into a possible cached version. Anyway, seems like is working correctly now.
Thank you for reviewing! I guess the issue can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants