Changeset 10876

Timestamp:

04/07/2021 03:51:17 AM (6 years ago)

Author:

dd32

Message:

Support Forums: Don't wrap pastes in code blocks when the paste cursor previous character is a backtick.

This also doesn't affect pastes when they begin OR end with a backtick, allowing for predefs which end with a code block but do not start with one.

See #2107.

File:

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

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

@@ -26 +26 @@
         $( '#new-post textarea' ).on( 'paste', function( e ) {
                 var $this = $(this),
-                        $val  = $this.val(),
-                        paste = ( e.originalEvent.clipboardData || window.clipboardData ).getData('text');
+                        val  = $this.val(),
+                        paste = ( e.originalEvent.clipboardData || window.clipboardData ).getData('text').trim();
 
                 // If no pasted text, or no textarea value, skip.
-                if ( ! paste.length || ! $val.length ) {
+                if ( ! paste.length || ! val.length ) {
                         return;
                 }
 
                 if (
-                        paste.length < 1000 &&        // Super long pastes get code wrapped
-                        paste.split("\n").length < 10 // in addition to many-lines pastes.
+                        paste.length < 500 &&        // Super long pastes get code wrapped
+                        paste.split("\n").length < 5 // in addition to many-lines pastes.
                 ) {
+                        return;
+                }
+
+                // See if the author is pasting into a code block already
+                if ( '`' === val.substr( $this.prop('selectionStart') - 1 , 1 ) ) {
+                        return;
+                }
+
+                // If the code being pasted is already wrapped in backticks (well, starts with OR ends with), skip.
+                if (
+                        '`' === paste.substr( 0, 1 ) ||
+                        '`' === paste.substr( -1, 1 )
+                ) { 
                         return;
                 }
         
                 $this.val(
-                        $val.substring( 0, $this.prop('selectionStart') ) +       // Text before cusor/selection
-                        "`" + paste.trim().replace(/^`|`$/g, '') + "`" +          // The pasted text, trimming ` off it and wrapping with `
-                        $val.substring( $this.prop('selectionEnd'), $val.length ) // Text after cursor position/selection
+                        val.substr( 0, $this.prop('selectionStart') ) +      // Text before cusor/selection
+                        "`" + paste + "`" +                                  // The pasted text, wrapping with `
+                        val.substr( $this.prop('selectionEnd'), val.length ) // Text after cursor position/selection
                 );