#42543 closed defect (bug) (fixed)
Minor type in class-wp-editor.php
Reported by: | GunGeekATX | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.1 | Priority: | normal |
Severity: | normal | Version: | 5.1 |
Component: | Editor | Keywords: | good-first-bug has-patch |
Focuses: | ui, docs | Cc: |
Description
There's a minor typo in the get_translation()
function, an extra space at the end of the 'Edit' field.
'Edit ' => __( 'Edit' ), // Tooltip for the 'edit' button in the image toolbar
Attachments (3)
Change History (18)
#5
in reply to:
↑ 4
@
7 years ago
Replying to SergeyBiryukov:
It's not a typo, see the comment in wpeditimage/plugin.js.
It does look weird though, maybe something like
Edit|button
could be used instead.
Thanks for the clarification on that @SergeyBiryukov! I Perhaps the fix is either to use 'Edit|button` as you suggested or at the very least, add an inline comment that has a similar comment and references the JS files?
#7
@
7 years ago
Here's a patch for updating the comment from wpeditimage/plugin.js. I briefly tried updating the edit to Edit|button
but it wasn't getting parsed correctly for me.
#8
@
7 years ago
Sorry for the second patch, just learned how to submit a patch. I think the first patch from @scottlee is better and simpler.
#10
in reply to:
↑ 9
@
7 years ago
Replying to SergeyBiryukov:
@scottlee has submitted the patch but the ticket is still labeled as 'needs patch' , do we still need a patch for this ?
#11
@
7 years ago
- Keywords needs-refresh has-patch added; needs-patch removed
Due to [42343] this patch needs a refresh.
It's not a typo, see the comment in wpeditimage/plugin.js.
It does look weird though, maybe something like
Edit|button
could be used instead.