#40582 closed enhancement (fixed)
Capitalize character - "CTRL + a" -> "CTRL + A"
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.0 | Priority: | normal |
Severity: | minor | Version: | 4.7.4 |
Component: | Administration | Keywords: | has-patch needs-user-docs |
Focuses: | ui-copy | Cc: |
Description
Attachments (5)
Change History (24)
#3
@
4 years ago
To match wp-includes/class-wp-editor.php, 40582.tinymce.diff has lowercase letters after the capital C: Ctrl + V
. Likewise, 40582.permalink.diff has Ctrl + A
, and it includes some small grammatical changes.
These patches are purposely separate because ticket:40763#comment:6 recommended the removal of Ctrl + A
and "we" wording (see also #46057). So I'll open a new ticket to explore more substantial Permalinks text changes (#53141) and to add label tags for each textarea (#53142).
#4
follow-up:
↓ 8
@
4 years ago
I don't think this change should be made.
It is confusing to see a capital letter because you need to use the shift key to get the capital. I have wondered about that in the past (not necessarily with this particular software).
And on my keyboard, it says "ctrl", not "Ctrl" or "CTRL".
#6
@
4 years ago
The proposed capitalization change is for consistency, but I don't have a strong opinion about it.
#7
@
4 years ago
I'm also not sure if the proposal is about capitalization or decapitalization. But yes, it's about consistency.
#8
in reply to:
↑ 4
@
4 years ago
- Keywords commit added
Replying to joyously:
It is confusing to see a capital letter because you need to use the shift key to get the capital.
That's a valid point. However, it looks like the patches here just make sure we use a consistent format for keyboard shortcuts in core, which I think is preferable to using an inconsistent format :) Whether the format itself should be changed, can be discussed in a new ticket.
And on my keyboard, it says "ctrl", not "Ctrl" or "CTRL".
On mine it's "Ctrl" :)
#9
@
3 years ago
- Keywords commit removed
- Milestone changed from Awaiting Review to 6.0
- Owner set to audrasjb
- Status changed from new to reviewing
#10
@
3 years ago
- Component changed from TinyMCE to Administration
- Keywords has-patch needs-refresh added
Moving to component "Administration" as the change is not only related to TinyMCE :)
This ticket was mentioned in PR #2364 on WordPress/wordpress-develop by audrasjb.
3 years ago
#11
- Keywords needs-refresh removed
Trac ticket: https://core.trac.wordpress.org/ticket/40582
3 years ago
#15
Committed in https://core.trac.wordpress.org/changeset/52808
#16
@
3 years ago
- Keywords commit removed
@sabernhardt regarding the use of "if it was"/"if it were", I started to open a new ticket to address this issue, but as I'm not a native English speaker I checked on the internet and I'm not 100% sure we need to change anything…
https://www.quora.com/When-do-you-use-if-it-were-and-if-it-was
Second patch is variant with Ctrl + A
(We have "CTRL + V" in wp-includes\js\tinymce\langs\wp-langs-en.js)