Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#3958 closed task (blessed) (fixed)

New style internationalization of quicktags.js

Reported by: nbachiyski Owned by: Nazgul
Milestone: 2.2 Priority: normal
Severity: normal Version: 2.2
Component: General Keywords: i18n js has-patch
Focuses: Cc:


quicktags.js should also be i18ned using the new system, devised by mdwaffe.

However we should think of other place for defining the translatable strings, not in script-loader.php, because there could be plenty of them. We can have a separate file for js gettext strings - script-i18n.php.

Attachments (2)

3958.diff (3.8 KB) - added by Nazgul 9 years ago.
3958b.diff (3.9 KB) - added by Nazgul 9 years ago.

Download all attachments as: .zip

Change History (10)

#1 @nbachiyski
9 years ago

  • Keywords i18n js added

#2 @Sewar
9 years ago

See also #3181

#3 @ryan
9 years ago

A separate file is fine.

9 years ago

#4 @Nazgul
9 years ago

  • Keywords has-patch added
  • Milestone changed from 2.3 to 2.2
  • Owner changed from anonymous to Nazgul

#5 @ryan
9 years ago

s/Quick Links1/Quick Links/

I think we should attribute_escape() strings that are bound for attributes.

Otherwise, looks good.

9 years ago

#6 @Nazgul
9 years ago

  • Status changed from new to assigned

The patch has been updated per ryan's suggestions.

I removed the 1 (wonder how it got in in the first place) and added attribute_escape() to the 4 strings that go into attributes.

#7 @ryan
9 years ago

We can handle moving the strings to another file with a new ticket.

#8 @ryan
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [5167]) i18n for quicktags.js. Props Nazgul. fixes #3958

Note: See TracTickets for help on using tickets.