-
Notifications
You must be signed in to change notification settings - Fork 14
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
Uppercase first character of a translation key #1
base: master
Are you sure you want to change the base?
Conversation
@@ -58,7 +58,7 @@ ckeditor.admin.reset.confirmation=Are you sure you want to restore the default c | |||
ckeditor.plugin.link.options=Options | |||
ckeditor.plugin.link.anchor=Anchor | |||
ckeditor.plugin.link.queryString=Query String | |||
ckeditor.plugin.link.defaultLabel=type the link label | |||
ckeditor.plugin.link.defaultLabel=Type the link label |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This text (and the link label in general) is inserted in-line, most probably in the middle of a sentence so I'm not convinced that it should start with an upper case letter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://snag.gy/ioyfVq.jpg See? "Find a page" with a uppercase and "type the link label" with lowercase but I understand what you mean ... You can reject my PR if you wish :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed https://snag.gy/ioyfVq.jpg needs to be fixed (either in code with a StringUtils.capitalize() or in transaltion).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question B: do we need to provide a default value? or can we just use a placeholder for the input?
- Depends on the answer to A, since if we add a link without a value, it will be hard for the user to know where the link starts/ends.
- A solution would be to make the "Display Text" mandatory, but this will break cases when link displays the page name if no link value is provided.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the same key is used for 2 different use cases then we should probably introduce a new key.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not for 2 different use cases, the key is used as value so it gets inserted inside the text.
No description provided.