Changeset 10669

Timestamp:

02/12/2021 05:05:46 AM (4 years ago)

Author:

dd32

Message:

Support Forums: Adjust the ajaxification of the view reply actions to not block open-in-new-tab and to only suffix a notice, rather than removing the content entirely.

See #1965.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/js/forums.js (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-support/js/forums.js

@@ -23 +23 @@
     }
 
-    if ( $( 'body' ).is( '.bbp-is-view' ) ) {
+    if ( $( 'body' ).is( '.bbp-view' ) ) {
         $( '.bbp-body .bbp-admin-links a' ).click( function( e ) {
             var $this = $( this ),
                 $element = $this.closest( '.bbp-body' ),
+                $content = $element.find( '.bbp-topic-content' ),
                 type;
+
+            // Don't affect open-in-new-tab.
+            if ( e.metaKey || e.ctrlKey || e.which === 2 ) {
+                return;
+            }
 
             if ( $this.is( '[class*=approve]' ) ) {
@@ -41 +47 @@
             e.preventDefault();
 
+            $element.fadeTo( 500, .5 );
+
             $.get( $this.prop( 'href' ) ).done( function() {
-                $element.fadeOut( 250, function() {
-                    $element.html(
-                        '<div class="bbp-template-notice">' + wporgSupport.strings[ type ] + '</div>'
-                    );
-                } ).fadeIn( 250 );
+                $content.append(
+                    '<div class="bbp-template-notice">' + wporgSupport.strings[ type ] + '</div>'
+                );
+
+                // Remove actions.
+                $this.parent().find('a:not(.bbp-topic-edit-link)').remove();
+
+                $element.fadeTo( 250, 1 );
             } ).error( function() {
-                $element.fadeOut( 250, function() {
-                    $element.find( '.bbp-topic-content' ).prepend(
-                        '<div class="bbp-template-notice">' + wporgSupport.strings.action_failed + '</div>'
-                    );
-                } ).fadeIn( 250 );
+                $content.append(
+                    '<div class="bbp-template-notice">' + wporgSupport.strings.action_failed + '</div>'
+                );
+
+                $element.fadeTo( 250, 1 );
             } );
         } );