[buddypress-trac] [BuddyPress Trac] #7840: bp_is_messages_compose_screen not working with BP Nouveau

buddypress-trac noreply at wordpress.org
Sun May 20 15:15:07 UTC 2018


#7840: bp_is_messages_compose_screen not working with BP Nouveau
--------------------------+---------------------
 Reporter:  sbrajesh      |       Owner:  (none)
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.0.1
Component:  Templates     |     Version:  3.0.0
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+---------------------
Changes (by imath):

 * keywords:  reporter-feedback =>


Comment:

 Thanks a lot for your reply @sbrajesh

 > Also, In my personal view, It will be a bad idea to have different
 behaviour for core functions based on template packs. Many plugin which
 still uses these to check for the compose screen will fail.

 imho this will also happen when the BP Rest API will be available ;)
 Although i think we need to move forward and try to improve the user
 experience, I totally understand your concern.

 The only way i can think of would be to break the Single Page App if a
 specific filter like `bp_nouveau_message_use_multi_page_app` is `true`. I
 can transport this into the page and do the needed edits into the Backbone
 router & views to open "legacy" urls instead of Backbone routes. Is this a
 good plan ?

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


More information about the buddypress-trac mailing list