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

Check for more than one capital letter per word #8753

Open
comradekingu opened this issue Feb 6, 2023 · 3 comments
Open

Check for more than one capital letter per word #8753

comradekingu opened this issue Feb 6, 2023 · 3 comments
Labels
enhancement Adding or requesting a new feature. good first issue Opportunity for newcoming contributors. help wanted Extra attention is needed.

Comments

@comradekingu
Copy link
Contributor

Describe the problem

THis is an error, and it often stems from holding down shift for too long, etc.

Describe the solution you'd like

A check triggered when a word contains more than one capital letter, which is an error in most any language (?)

Describe alternatives you've considered

If someone wants to do "ALL CAPS", or "CamelCase" in user-facing strings I think it follows that it is most often dumb enough to where having to add "ignore-doublecapital" flags for the ones that actually make sense is warranted.

Screenshots

No response

Additional context

Taken from #5281

@nijel nijel added enhancement Adding or requesting a new feature. help wanted Extra attention is needed. good first issue Opportunity for newcoming contributors. labels Feb 6, 2023
@github-actions
Copy link

github-actions bot commented Feb 6, 2023

This issue seems to be a good fit for newbie contributors. You are welcome to contribute to Weblate! Don't hesitate to ask any questions you would have while implementing this.

You can learn about how to get started in our contributors documentation.

@tjhietala
Copy link
Contributor

There are lots of names like DeLorean, McFly, O'Brien, etc. with more than one capital letter per word. And in computing: SmartArt, JavaScript, GitHub (!), etc.

You could of course look for two consecutive capital letters. But no more than two, unless you want to get false positives for things like PDF, SQL, HTML, etc.

@nijel
Copy link
Member

nijel commented Feb 27, 2023

The check should work the same as others – anything what is present in the source string is considered okay, it should trigger only on capitalization added in the translation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adding or requesting a new feature. good first issue Opportunity for newcoming contributors. help wanted Extra attention is needed.
Projects
None yet
Development

No branches or pull requests

3 participants