[wp-trac] [WordPress Trac] #47161: Do not break strings to avoid HTML tags in them

WordPress Trac noreply at wordpress.org
Fri May 17 00:02:20 UTC 2019


#47161: Do not break strings to avoid HTML tags in them
--------------------------+-----------------------------
 Reporter:  dimadin       |       Owner:  SergeyBiryukov
     Type:  defect (bug)  |      Status:  reviewing
 Priority:  normal        |   Milestone:  5.2.1
Component:  I18N          |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:
--------------------------+-----------------------------

Comment (by SergeyBiryukov):

 Replying to [comment:2 Clorith]:
 > The link location is actually intentional here. The link is included for
 context in the description on where the requirements and recommendations
 come from, and belongs as part of the description.
 >
 > The action section is for actionable items to resolve a failing test,
 which this link is not.

 My initial thought was that this link isn't much different from "Learn
 more about updating PHP" and "Read more about why you should use HTTPS",
 which are both placed in the action section.

 On second thought, I see the difference: the latter two are user-oriented
 and located in HelpHub, while the link in question is a more technical-
 oriented list of recommended PHP extensions in the Hosting team handbook.

 If we had a HelpHub article on managing PHP extensions, that would be a
 better fit for the action section. For now, let's just fix the i18n best
 practices regression in [45170].

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


More information about the wp-trac mailing list