WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#33777 closed defect (bug) (fixed)

Duplicate string - Open link in a new tab

Reported by: pavelevap Owned by: SergeyBiryukov
Priority: normal Milestone: 4.4
Component: I18N Version: 4.4
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

We have two duplicate strings:

Open link in a new tab (Menu Customizer)

Open link in a new window/tab (Menu page, Link modal)

I can create patch, but I am not sure which one is better? First one was created for Menus in Customizer, second is older. But there is no reason to have both, I guess...

Attachments (2)

33777.diff​ (744 bytes ) - added by dipesh.kakadiya 11 years ago.
Updated string as 'Open link in a new window/tab'. As I check code every where given strign is used.
33777_1.diff​ (2.3 KB ) - added by dipesh.kakadiya 11 years ago.
Update short string Open link in a new tab

Download all attachments as: .zip

Change History (6)

@dipesh.kakadiya
11 years ago

Updated string as 'Open link in a new window/tab'. As I check code every where given strign is used.

#1 @dipesh.kakadiya
11 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
11 years ago

  • Milestone Awaiting Review → 4.4

Let's go with the first one, it's hard to find a browser without tabs these days :)

#3 @pavelevap
11 years ago

I agree, shorter string was probably created to better fit in Customizer...

@dipesh.kakadiya
11 years ago

Update short string Open link in a new tab

#4 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 33998:

Merge two similar strings.

Props dipesh.kakadiya.
Fixes #33777.

Note: See TracTickets for help on using tickets.