WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#31844 closed defect (bug) (fixed)

Translation strings merge: edit site in the network

Reported by: ramiy Owned by: SergeyBiryukov
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.2
Component: I18N Keywords:
Focuses: administration, multisite Cc:

Description

Ok, I found more strings that can be merged.

By moving the HTML out, we can merge those strings, making the PO/MO file smaller.

Attachments (1)

31844.patch (3.0 KB) - added by ramiy 5 years ago.

Download all attachments as: .zip

Change History (5)

@ramiy
5 years ago

#1 @ramiy
5 years ago

Before:

  • 4 strings - 'Edit Site: <a href="%1$s">%2$s</a>'
  • 4 strings - 'Edit Site: %s'

After:

  • 8 strings - 'Edit Site: %s'

#2 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 4.2

#3 @SergeyBiryukov
5 years ago

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

In 31962:

Merge title strings on Edit Site screens.

props ramiy.
fixes #31844.

#4 @SergeyBiryukov
5 years ago

  • Component changed from Networks and Sites to I18N
Note: See TracTickets for help on using tickets.