[wp-trac] [WordPress Trac] #43434: Add sprintf function and target attribute in href

[WordPress Trac]

#43434: Add sprintf function and target attribute in href
-------------------------------------+----------------------------
 Reporter:  mukesh27                 |       Owner:
     Type:  enhancement              |      Status:  closed
 Priority:  normal                   |   Milestone:
Component:  General                  |     Version:  4.9.4
 Severity:  normal                   |  Resolution:  wontfix
 Keywords:  i18n-change 2nd-opinion  |     Focuses:  accessibility
-------------------------------------+----------------------------
Changes (by mukesh27):

 * keywords:  2nd-opinion => i18n-change 2nd-opinion
 * focuses:  administration => accessibility
 * component:  Users => General


Comment:

 * We have to change that link as it's part of coding standard.
    * Yes, right now when user click on that help artical then that page
 open in current tab so needs to open in new tab so user can see that
 document and not lost there current page.
    * Yes but we have to add sprintf so all Locales can replace string if
 they want.

 Replying to [comment:3 audrasjb]:
 > Hello @mukesh27
 >
 > Thanks for this ticket.
 > However, in my opinion:
 > - Similar links are used in many help panels, like `widgets.php` for
 example. Any change on `users.php` should be consistent in other screens.
 > - Target blank is very often a bad idea. At least, it should be
 identified as "opens-in-a-new-window" links.
 > - I do not think using sprintf is relevant as some Locales can replace
 the links with their own translated resources.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/43434#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform