[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