#40582 closed enhancement (fixed)
Capitalize character - "CTRL + a" -> "CTRL + A"
Reported by: | Presskopp | Owned by: | audrasjb |
---|---|---|---|
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
@
3 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
@
3 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
@
3 years ago
The proposed capitalization change is for consistency, but I don't have a strong opinion about it.
#7
@
3 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
@
3 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.
It's also worth noting that keys on the keyboard itself generally use capital letters.
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)