[buddypress-trac] [BuddyPress Trac] #8079: BP Site-wide Notices: To enable paragraphs in the body text of the notification.

[buddypress-trac]

#8079: BP Site-wide Notices: To enable paragraphs in the body text of the
notification.
-------------------------------------------------+---------------------
 Reporter:  kristianngve                         |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  5.0.0
Component:  Messages                             |     Version:  4.2.0
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch 2nd-opinion needs-testing  |
-------------------------------------------------+---------------------
Changes (by imath):

 * keywords:  reporter-feedback => has-patch 2nd-opinion needs-testing
 * milestone:  Awaiting Review => 5.0.0


Comment:

 Thanks for your feedback @kristianngve

 So here's what I'm suggesting in 8079.patch:
 1. Use the notice template tags inside `bp_message_get_notices()` to
 output the subject and text of the notice.
 2. Remove extra the `p` tag containing the title and text as the text will
 be `wpautop` thanks to the `bp_message_notice_text` template tag.
 3. Adapt the Legacy styles to the new markup.
 4. This will let users do `remove_filter( 'bp_message_notice_text',
 'wp_filter_kses' )` and choose the kses filter they prefer (eg:
 `add_filter( 'bp_message_notice_text', 'bp_messages_filter_kses' )`.

 NB: The problem in `bp_message_get_notices()` is that `wp_filter_kses` is
 not applied as a filter.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8079#comment:6>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac