Context Navigation

← Previous Changeset
Next Changeset →

Changeset 10876

Timestamp:

04/07/2021 03:51:17 AM (5 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:

: 1 edited

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

Legend:

: Unmodified
: Added
: Removed

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

-              r10702
+              r10876
         $( '#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
                 );

Note: See TracChangeset for help on using the changeset viewer.

Making WordPress.org

Context Navigation

Changeset 10876

Legend:

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

Download in other formats: