[buddypress-trac] [BuddyPress Trac] #6712: Screen notifications settings page

buddypress-trac noreply at wordpress.org
Wed Apr 20 21:25:14 UTC 2016


#6712: Screen notifications settings page
---------------------------------------+------------------
 Reporter:  r-a-y                      |       Owner:
     Type:  enhancement                |      Status:  new
 Priority:  normal                     |   Milestone:  2.6
Component:  Component - Notifications  |     Version:  1.9
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch early            |
---------------------------------------+------------------

Comment (by r-a-y):

 > But my first comment would be: the src/bp-templates/bp-
 legacy/buddypress/members/single/settings/screen-notifications.php really
 needs to be less scary!! There's some inline PHP code and inline
 Javascript code. Is there a way to avoid it ?

 Yeah, I know the template is a little rough with regards to inline PHP at
 the moment.

 See comment:10, specifically:

 > The template is still rough around the edges code-wise, but I'm waiting
 for feedback on the look and feel before iterating further.

 I'm thinking about moving the `table` code to a template tag.  I'll work
 on this a bit later on so it isn't so scary :)

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


More information about the buddypress-trac mailing list