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

Translate to english #269

Merged
merged 83 commits into from
Dec 1, 2023
Merged

Translate to english #269

merged 83 commits into from
Dec 1, 2023

Conversation

vvatelot
Copy link
Member

No description provided.

@bgatellier
Copy link
Contributor

bgatellier commented Nov 21, 2023

@yaaax Le commit 0b64876 active la redirection de l'utilisateur vers la page de résultat localisée 🎉

La page de résultat anglaise comporte encore pas mal de contenu en français, et la majorité sont situé dans les partials (partials/widgets notamment) de Hugo. Je ne suis pas sûr de savoir quelle est la bonne manière de traduire ces contenus, est-ce que tu peux stp :

  • soit jeter un oeil et le mettre en place
  • soit m'expliquer comment m'y prendre (un lien vers une ou plusieurs docs peuvent suffirent)

Merci !

@bgatellier bgatellier added the enhancement New feature or request label Nov 22, 2023
@bgatellier
Copy link
Contributor

Hello @vvatelot ,

Je pense avoir fait le tour des traductions sur toutes les pages.
Si tu peux retester l'ensemble ce serait 👍

@vvatelot
Copy link
Member Author

Pour moi c'est top !
Je me suis juste permis de mettre à jour le lien vers mon profil ;)

@vvatelot
Copy link
Member Author

vvatelot commented Nov 24, 2023

Étrange, il semble qu'il yaie un décalage de mise en page:
Nouvelle version
image

VS Version originale
image

@bgatellier
Copy link
Contributor

Étrange, il semble qu'il yaie un décalage de mise en page: Nouvelle version image

VS Version originale image

Ce n'est pas illusion due à la présence du switcher de langue en haut à droite ?
Car je ne vois pas sinon où il y aurait un problème; tu peux encercler de rouge sur le screenshot ?

@bgatellier bgatellier linked an issue Nov 24, 2023 that may be closed by this pull request
@vvatelot
Copy link
Member Author

Pour que ce soit plus explicite, voilà une superposition des 2 screenshot... :)

ecoindex

@bgatellier
Copy link
Contributor

bgatellier commented Nov 24, 2023

Pour que ce soit plus explicite, voilà une superposition des 2 screenshot... :)

ecoindex

C'était du à la mise à jour du module base-structure dans ce commit @yaaax
Laquelle mise à jour n'est pas rétrocompatible.
Du coup j'ai revert et tout semble rentré dans l'ordre @vvatelot

@vvatelot
Copy link
Member Author

Hum... 🤔 maintenant suite au rollback, il y a le bug du doublon:
image

@yaaax ça te parle tout ça ?

@bgatellier
Copy link
Contributor

CSS corrigés @vvatelot

@vvatelot
Copy link
Member Author

vvatelot commented Dec 1, 2023

ça me semble tout bon !
J'ai déployé en preprod et c'est OK pour moi !

@vvatelot
Copy link
Member Author

vvatelot commented Dec 1, 2023

Par contre, je ne peux pas valider cette PR comme c'est moi qui l'avait initié... 🙂

@bgatellier bgatellier self-requested a review December 1, 2023 15:58
@bgatellier bgatellier merged commit d75c87a into master Dec 1, 2023
3 checks passed
@bgatellier bgatellier deleted the translate-to-english branch December 1, 2023 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create English content
6 participants