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

Capitalise analysis results in source strings instead of CSS #16731

Merged
merged 3 commits into from
Jan 6, 2025

Conversation

superuser-does
Copy link
Collaborator

@superuser-does superuser-does commented Jan 5, 2025

Capitalised the first three entries of the computer analysis results table in source strings instead of through CSS, per discussion in #16649.

In the absence of a convention, I changed nbFoo to numberFoo for the new strings. The key must be different, and I hope this is a suitable substitute.

  • I believe this will still build based on the CI (I have not built it myself).
  • This commit should not delete the old strings on Crowdin because they are in the XML file under translation/source. Crowdin is only concerned with the translation folder, which is why I think it's safe to remove the corresponding entries for the old keys in i18n.d.ts and key.scala.

Once pulled, this commit will enable me to run pre-translation on the Crowdin side for the new string.
At that point, I can make a separate commit to remove the old strings.

New keys introduced; the existing keys will be retained in /translation/source until Crowdin pre-translation is complete.
The old keys should be deleted only after the pre-translations have been pulled into master.
@ornicar ornicar merged commit 78e28df into master Jan 6, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants