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

Publish du html a une liste de mots (partie 1) #3020

Merged
merged 6 commits into from
Sep 14, 2023

Conversation

marie-flesch
Copy link
Contributor

I am publishing the lesson "Du HTML à une liste de mots (partie 1)

changing "translation reviewers" to "translation reviewer"
Copy link
Contributor

@spapastamkou spapastamkou left a comment

Choose a reason for hiding this comment

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

faudrait peut-être descendre le niveau des titres de 1 à chaque fois: les titres niveau 1 => niveau 2 et ceux de niveau 2 => niveau 3 (pour harmoniser avec les autres versions de la leçon en EN, PT et ES). Sinon cela a l'air top!

@marie-flesch
Copy link
Contributor Author

faudrait peut-être descendre le niveau des titres de 1 à chaque fois: les titres niveau 1 => niveau 2 et ceux de niveau 2 => niveau 3 (pour harmoniser avec les autres versions de la leçon en EN, PT et ES). Sinon cela a l'air top!

Bien vu, merci ! Oui on va faire ça. En fait finalement il y a d'autres changements à apporter, pour qu'il y ait une cohérence entre la partie 1 et la partie 2 (qui est en cours d'évaluation). Célian (le traducteur) s'en occupe la semaine prochaine !

@datalogism
Copy link
Contributor

Hello @marie-flesch c'est tout bon pour moi :)
Bien repris ton commentaire @spapastamkou, j'ai aussi corrigé cela sur la partie 2 !

@anisa-hawes
Copy link
Contributor

Hello @marie-flesch. I've integrated Célian's most recent edits programminghistorian/ph-submissions@bbc78e9 and brought this branch up-to-date with gh-pages.

I'm unclear about how to mark the changes requested by Sofia as resolved so that we can merge... I'll investigate, and update you as soon as possible.

@marie-flesch
Copy link
Contributor Author

Thank you so much @anisa-hawes, it looks good!

@spapastamkou spapastamkou self-requested a review September 14, 2023 15:07
@spapastamkou
Copy link
Contributor

@anisa-hawes resolved :-) - thanks!

@marie-flesch marie-flesch merged commit cc8291b into gh-pages Sep 14, 2023
5 checks passed
@marie-flesch marie-flesch deleted the Publish-Du-HTML-a-une-liste-de-mots-(partie-1) branch September 14, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants