Changeset 12116 for branches/3.0/src/bp-templates/bp-nouveau/buddypress/common/js-templates/messages/index.php

Timestamp:

05/24/2018 05:17:07 AM (8 years ago)

Author:

imath

Message:

BP Nouveau Messages UI: Improve the Backbone router using pushState

In order to be consistent with conditional template tags such as bp_is_messages_compose_screen(), the routes of the Backbone router have been updated to match Messages component canonical URLs

Props sbrajesh, djpaul

Fixes #7840 (Branch 3.0)

File:

• branches/3.0/src/bp-templates/bp-nouveau/buddypress/common/js-templates/messages/index.php (modified) (2 diffs)

branches/3.0/src/bp-templates/bp-nouveau/buddypress/common/js-templates/messages/index.php

@@ -155 +155 @@
         <div class="thread-subject">
             <span class="thread-count">({{data.count}})</span>
-            <a class="subject" href="#view/{{data.id}}">{{data.subject}}</a>
+            <a class="subject" href="../view/{{data.id}}/">{{data.subject}}</a>
         </div>
         <p class="excerpt">{{data.excerpt}}</p>
@@ -204 +204 @@
                     <# } #>
 
-                    <a href="#view/{{data.id}}" class="message-action-view bp-tooltip bp-icons" data-bp-tooltip="<?php esc_attr_e( 'View full conversation and reply.', 'buddypress' ); ?>">
+                    <a href="../view/{{data.id}}/" class="message-action-view bp-tooltip bp-icons" data-bp-action="view" data-bp-tooltip="<?php esc_attr_e( 'View full conversation and reply.', 'buddypress' ); ?>">
                         <span class="bp-screen-reader-text"><?php esc_html_e( 'View full conversation and reply.', 'buddypress' ); ?></span>
                     </a>