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

GH-53: Add localization support (Resolves #53, #77, #80) #85

Merged
merged 44 commits into from Oct 7, 2021

Conversation

jobara
Copy link
Member

@jobara jobara commented Sep 29, 2021

Resolves #53
Resolves #77
Resolves #80

@jhung
Copy link
Member

jhung commented Sep 29, 2021

@jobara - can we add a contributing.md file which describes the process for adding and localizing content (and thus removing it from the readme file)? Ultimately it should help someone from the community contribute translations.

In particular I'd be looking for:

  • a step-by-step breakdown of how to do translations / how would I add new languages.
  • what happens if a given page has not been translated - what happens?
  • how is navigation handled? does the translator need to worry about this?
  • what kind of content should be translated (i.e. not just visible text, but also images, alt-text, aria-labels, etc.)

Reprendre le Guide à différentes étapes de votre projet peut être une bonne façon de l'intégrer à votre travail.

</figcaption>
<span class="small-12 medium-6 large-4 column"> {% include "svg/schematic.svg" %} </span>
Copy link
Member

Choose a reason for hiding this comment

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

The aria label for this SVG uses the <title> and <description> elements from the SVG which needs to be translated.

I think the aria-label should be generated another way.

Comment on lines +41 to +42
*Illustration 2 :* Un participant présente le concept de son groupe qui avait pour tâche de créer une version
informatique du circuit de billes.
Copy link
Member

@jhung jhung Sep 30, 2021

Choose a reason for hiding this comment

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

When rendered, a <br/> is injected between the lines causing a break. this happens with all captions.

image

This was never a problem because we never linted these md files before. :)

Copy link
Member Author

@jobara jobara Oct 4, 2021

Choose a reason for hiding this comment

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

This is caused by fluid-project/eleventy-plugin-fluid#76. I'll update the plugin once it has been addressed.

Copy link
Member

@jhung jhung left a comment

Choose a reason for hiding this comment

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

Done reviewing for now.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@jobara jobara changed the title GH-53: Add localization support (Resolves #53, #77) GH-53: Add localization support (Resolves #53, #77, #80) Oct 4, 2021
src/assets/styles/app.styl Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated
Comment on lines 52 to 53
1. Update the `Language`
2. Update the localized text
Copy link
Member

Choose a reason for hiding this comment

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

@jobara can you give a little more detail here?

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@jobara jobara requested a review from jhung October 5, 2021 16:55
@jobara
Copy link
Member Author

jobara commented Oct 5, 2021

@jhung ready for more review.

Copy link
Member

@jhung jhung left a comment

Choose a reason for hiding this comment

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

Just a minor change for consistency. Everything else looks good!

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Co-authored-by: jhung <jhung@ocadu.ca>
@jobara
Copy link
Member Author

jobara commented Oct 7, 2021

@jhung I merged your latest suggestions.

@jobara jobara merged commit 9dbc40e into inclusive-design:main Oct 7, 2021
@jobara
Copy link
Member Author

jobara commented Oct 7, 2021

merged at 9dbc40e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants