Skip to content

Improve some French localizations#129

Merged
lhecker merged 1 commit intomicrosoft:mainfrom
dan-jac:frenchLocFixes
May 20, 2025
Merged

Improve some French localizations#129
lhecker merged 1 commit intomicrosoft:mainfrom
dan-jac:frenchLocFixes

Conversation

@dan-jac
Copy link
Contributor

@dan-jac dan-jac commented May 20, 2025

This PR adresses some issues with the French localization of Edit. Specifically:

  • The Edit menu previously got translated to "Éditer", which does not match convention. In French, these menu titles are not traditionally in verb form; they're nouns. So Edit is usually translated to "Édition", which is what I've done here.
  • The existing string for "Focus statusbar" ("Focus sur la barre d'état") isn't great. This is a command, so it should be a verb in its infinitive form. That could be "focuser", but that's not really a French word; it's an anglicism that's been Frenchified a bit. I've changed this to "Activer", which matches convention and sounds natural.
  • There was inconsistency in the use of the "straight apostrophe" ' vs. the "true apostrophe" . I've updated the strings to consistently use the true apostrophe.
  • In French, there should be a space (non-breaking) before characters like question marks and colons. I've added these in, to match Microsoft's French style guide and other Microsoft apps.

Copy link
Member

@lhecker lhecker left a comment

Choose a reason for hiding this comment

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

I completely trust you on this. I'm not a native speaker and these were all machine translated. Thank you for doing this!

(The alternative is using Microsoft internal translation services, but this would've required me to use their services exclusively and reject your PR. I didn't want that.)

@lhecker lhecker merged commit 3ac343a into microsoft:main May 20, 2025
1 check passed
diabloproject pushed a commit to diabloproject/edit that referenced this pull request May 29, 2025
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.

2 participants