-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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.