#42543 closed defect (bug) (fixed)
Minor type in class-wp-editor.php
| Reported by: | GunGeekATX | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Editor | Version: | 5.1 |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: | ui, docs |
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
@
9 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|buttoncould 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
@
9 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
@
9 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
@
9 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 ?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
It's not a typo, see the comment in wpeditimage/plugin.js.
It does look weird though, maybe something like
Edit|buttoncould be used instead.