Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#45788 closed defect (bug) (fixed)

Not translated: "add media" in block classic in WP 5.0.2

Reported by: La Geek Owned by: desrosj
Priority: normal Milestone: 5.0.3
Component: Editor Version: 5.0.2
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

Although the translation is on 100% the icon "add media" is not translated. Seen in Block Classic.

Attachments (1)

45788.diff (922 bytes ) - added by afercia 8 years ago.

Download all attachments as: .zip

Change History (10)

#1 @swissspidy
8 years ago

  • Component I18NEditor
  • Keywords needs-patch added

@afercia
8 years ago

#2 @afercia
8 years ago

  • Keywords needs-patch removed
  • Milestone Awaiting Review5.0.3

45788.diff makes the tooltip string translatable.
Also, adds the keyboard shortcut to be displayed in the tooltip once https://github.com/WordPress/gutenberg/issues/6195 will be addressed.

#3 @afercia
8 years ago

  • Keywords has-patch added

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


8 years ago

#5 @audrasjb
8 years ago

  • Keywords commit added

Hi @afercia the patch works well on my side. Adding commit keyword for 5.0.3.

#6 @desrosj
8 years ago

  • Owner set to desrosj
  • Resolutionfixed
  • Status newclosed

In 44398:

Editor: Allow the “Add Media” button tooltip to be translated in the Classic block.

In the Classic Editor, the “Add Media” button was outside of the TinyMCE editor. In the Block Editor, the “Add Media” button was moved into the TinyMCE toolbar in the Classic block, but the tooltip was not added to the list of translatable strings. This adds “Add Media” to that list.

The corresponding keyboard shortcut is also specified for the “Add Media” button.

Props afercia.

Fixes #45788.

#7 @desrosj
8 years ago

  • Keywords commit removed
  • Resolution fixed
  • Status closedreopened

Reopening for backport.

#8 @desrosj
8 years ago

In 44399:

Editor: Fix name of the new editor for inline comment.

In [44398], an inline comment was introduced that did not refer to the new editor as the Block Editor. This corrects that comment.

See #45788.

#9 @desrosj
8 years ago

  • Resolutionfixed
  • Status reopenedclosed

In 44400:

Editor: Allow the “Add Media” button tooltip to be translated in the Classic block.

In the Classic Editor, the “Add Media” button was outside of the TinyMCE editor. In the Block Editor, the “Add Media” button was moved into the TinyMCE toolbar in the Classic block, but the tooltip was not added to the list of translatable strings. This adds “Add Media” to that list.

The corresponding keyboard shortcut is also specified for the “Add Media” button.

Props afercia.

Merges [44398-44399] to the 5.0 branch.
Fixes #45788.

Note: See TracTickets for help on using tickets.