WordPress.org

Make WordPress Core

Opened 7 months ago

Closed 5 months ago

#42543 closed defect (bug) (fixed)

Minor type in class-wp-editor.php

Reported by: GunGeekATX Owned by: SergeyBiryukov
Milestone: 5.0 Priority: normal
Severity: normal Version: trunk
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)

42543.diff (1.0 KB) - added by scottlee 7 months ago.
42543.2.diff (1.0 KB) - added by ayubadiputra 7 months ago.
42543.3.diff (1.2 KB) - added by NathanAtmoz 7 months ago.

Download all attachments as: .zip

Change History (16)

#1 @welcher
7 months ago

  • Keywords needs-patch good-first-bug added

#2 @welcher
7 months ago

  • Milestone changed from Awaiting Review to 4.9.1

#3 @welcher
7 months ago

  • Milestone changed from 4.9.1 to Future Release

#4 follow-up: @SergeyBiryukov
7 months ago

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.

#5 in reply to: ↑ 4 @welcher
7 months 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?

#6 @welcher
7 months ago

  • Focuses ui docs added

#7 @scottlee
7 months 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.

@scottlee
7 months ago

#8 @ayubadiputra
7 months ago

Sorry for the second patch, just learned how to submit a patch. I think the first patch from @scottlee is better and simpler.

#9 follow-up: @SergeyBiryukov
7 months ago

  • Milestone changed from Future Release to 5.0

#10 in reply to: ↑ 9 @XpertOne
7 months 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 @welcher
7 months ago

  • Keywords needs-refresh has-patch added; needs-patch removed

Due to [42343] this patch needs a refresh.

@NathanAtmoz
7 months ago

#12 @NathanAtmoz
5 months ago

  • Keywords needs-refresh removed

#13 @SergeyBiryukov
5 months ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 42523:

Editor: Use explicit context instead of a trailing space for Edit string in TinyMCE plugins.

Props scottlee, ayubadiputra, NathanAtmoz.
Fixes #42543.

Note: See TracTickets for help on using tickets.