[wp-trac] [WordPress Trac] #44082: settings_errors() wraps all notices in p and strong tags

WordPress Trac noreply at wordpress.org
Mon Nov 30 06:48:48 UTC 2020


#44082: settings_errors() wraps all notices in p and strong tags
-------------------------------------+------------------------------
 Reporter:  johnjamesjacoby          |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  General                  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:
-------------------------------------+------------------------------
Changes (by sabernhardt):

 * keywords:   => has-patch needs-testing


Comment:

 [attachment:"44082.diff"] follows the proposed change of using `wpautop`,
 to see if the simplest method is good. I've done a quick test so far, but
 I haven't tried to break it.

 I also thought of adding another parameter to the function ($formatted =
 false) to keep the current markup by default.

 #38734 already reported how core is not using the Settings API (and my
 search of the current trunk found 89 `is-dismissible` notices in wp-
 admin).

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


More information about the wp-trac mailing list