Skip to content

Improve some French localizations#129

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

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

Conversation

@dan-jac

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

Copy link
Copy Markdown
Contributor

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.

@lhecker lhecker left a comment

Copy link
Copy Markdown
Member

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
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