Skip to content

LG-15754: French LQA update#11911

Merged
mdiarra3 merged 11 commits intomainfrom
LG-15754-sprint-331-lqa
Feb 25, 2025
Merged

LG-15754: French LQA update#11911
mdiarra3 merged 11 commits intomainfrom
LG-15754-sprint-331-lqa

Conversation

@mdiarra3
Copy link
Contributor

@mdiarra3 mdiarra3 commented Feb 24, 2025

🎫 Ticket

Link to the relevant ticket:
LG-15754

🛠 Summary of changes

Update French Language page for webauthn setup mismatch to so << >> is in one line.

👀 Screenshots

Before
Screenshot 2025-02-24 at 12 46 38 PM

After
Screenshot 2025-02-24 at 2 51 20 PM (2)

@mdiarra3 mdiarra3 requested a review from a team February 24, 2025 17:48
Comment on lines +2049 to +2050
webauthn_setup_mismatch.description.webauthn_html: Nous avons remarqué que vous utilisiez une clé de sécurité au lieu du déverrouillage facial ou tactile. Cliquez sur «&nbsp;Suite » pour utiliser votre clé de sécurité afin de vous connecter à partir de maintenant.
webauthn_setup_mismatch.description.webauthn_platform_html: Nous avons remarqué que vous utilisiez le déverrouillage facial ou tactile au lieu d’une clé de sécurité. Cliquez sur «&nbsp;Suite » pour utiliser le déverrouillage facial ou tactile afin de vous connecter à partir de maintenant.
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll want &nbsp; on both ends of the text, to prevent orphaned closing quotemark.

Suggested change
webauthn_setup_mismatch.description.webauthn_html: Nous avons remarqué que vous utilisiez une clé de sécurité au lieu du déverrouillage facial ou tactile. Cliquez sur «&nbsp;Suite » pour utiliser votre clé de sécurité afin de vous connecter à partir de maintenant.
webauthn_setup_mismatch.description.webauthn_platform_html: Nous avons remarqué que vous utilisiez le déverrouillage facial ou tactile au lieu d’une clé de sécurité. Cliquez sur «&nbsp;Suite » pour utiliser le déverrouillage facial ou tactile afin de vous connecter à partir de maintenant.
webauthn_setup_mismatch.description.webauthn_html: Nous avons remarqué que vous utilisiez une clé de sécurité au lieu du déverrouillage facial ou tactile. Cliquez sur «&nbsp;Suite&nbsp;» pour utiliser votre clé de sécurité afin de vous connecter à partir de maintenant.
webauthn_setup_mismatch.description.webauthn_platform_html: Nous avons remarqué que vous utilisiez le déverrouillage facial ou tactile au lieu d’une clé de sécurité. Cliquez sur «&nbsp;Suite&nbsp;» pour utiliser le déverrouillage facial ou tactile afin de vous connecter à partir de maintenant.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hmm should we do this for all french translations with that quotation format?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I brought it up when we refined the ticket (and partially captured in Charlie's comment on the ticket) that it would be a good idea to enforce this consistently.

Comment on lines +2049 to +2050
webauthn_setup_mismatch.description.webauthn_html: Nous avons remarqué que vous utilisiez une clé de sécurité au lieu du déverrouillage facial ou tactile. Cliquez sur « Suite » pour utiliser votre clé de sécurité afin de vous connecter à partir de maintenant.
webauthn_setup_mismatch.description.webauthn_platform_html: Nous avons remarqué que vous utilisiez le déverrouillage facial ou tactile au lieu d’une clé de sécurité. Cliquez sur « Suite » pour utiliser le déverrouillage facial ou tactile afin de vous connecter à partir de maintenant.
Copy link
Contributor

Choose a reason for hiding this comment

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

I expect the tests will flag it, but this won't need _html suffix anymore now that we've removed the HTML entity.

Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@aduth
Copy link
Contributor

aduth commented Feb 25, 2025

The lint failure for character glyphs looks unexpected. I can take a look since I worked on that scraping tool.

@aduth
Copy link
Contributor

aduth commented Feb 25, 2025

Actually, I think it is expected, since it looks like we've removed all instances of (\u2009, "thin space"). It looks like this was being used in a couple strings (e.g. notices.timeout_warning.partially_signed_in.live_region_message_html). I doubt we'd intentionally used it, or that it'd be a problem to use the non-breaking space instead to standardize.

In which case, we can just commit the changes to glyphs.txt after running make lint_font_glyphs and it should make the error go away.

@mdiarra3 mdiarra3 merged commit 6606e0f into main Feb 25, 2025
2 checks passed
@mdiarra3 mdiarra3 deleted the LG-15754-sprint-331-lqa branch February 25, 2025 19:00
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.

4 participants