WordPress.org
Get WordPress

Make WordPress Core

Ticket #31823: 31823.2.diff

File 31823.2.diff, 6.2 KB (added by adamsilverstein, 7 years ago)

  • Gruntfile.js 

    diff --git a/Gruntfile.js b/Gruntfile.js
index c7e7f73b2b..ec9b84a24a 100644
    a b  
    1 /* jshint node:true */
     1/* global module, require, __dirname */
    22module.exports = function(grunt) {
    33        var path = require('path'),
    44                fs = require( 'fs' ),
    … … module.exports = function(grunt) { 
    130130                                                        version = version.replace( /-[\d]{5}$/, '-' + grunt.template.today( 'yyyymmdd.HHMMss' ) );
    131131
    132132                                                        /* jshint quotmark: true */
    133                                                         return "$wp_version = '" + version + "';";
     133                                                        return '$wp_version = \'' + version + '\';';
    134134                                                });
    135135                                        }
    136136                                },
    … … module.exports = function(grunt) { 
    292292                                src: []
    293293                        }
    294294                },
     295                eslint: {
     296                        grunt: {
     297                                options: {
     298                                        configFile: 'node_modules/eslint-config-wordpress/index.js'
     299                                },
     300                                src: ['Gruntfile.js']
     301                        },
     302                        jshint: {
     303                                options: {
     304                                        configFile: 'node_modules/eslint-config-wordpress/index.js'
     305                                },
     306                                cwd: SOURCE_DIR,
     307                                src: [
     308                                        'wp-admin/js/*.js',
     309                                        'wp-includes/js/*.js',
     310                                        // Built scripts.
     311                                        '!wp-includes/js/media-*',
     312                                        // WordPress scripts inside directories
     313                                        'wp-includes/js/jquery/jquery.table-hotkeys.js',
     314                                        'wp-includes/js/mediaelement/wp-mediaelement.js',
     315                                        'wp-includes/js/mediaelement/wp-playlist.js',
     316                                        'wp-includes/js/plupload/handlers.js',
     317                                        'wp-includes/js/plupload/wp-plupload.js',
     318                                        'wp-includes/js/tinymce/plugins/wordpress/plugin.js',
     319                                        'wp-includes/js/tinymce/plugins/wp*/plugin.js',
     320                                        // Third party scripts
     321                                        '!wp-admin/js/farbtastic.js',
     322                                        '!wp-includes/js/backbone*.js',
     323                                        '!wp-includes/js/swfobject.js',
     324                                        '!wp-includes/js/underscore*.js',
     325                                        '!wp-includes/js/colorpicker.js',
     326                                        '!wp-includes/js/hoverIntent.js',
     327                                        '!wp-includes/js/json2.js',
     328                                        '!wp-includes/js/tw-sack.js',
     329                                        '!wp-includes/js/twemoji.js',
     330                                        '!**/*.min.js'
     331                                ]
     332                        },
     333                        themes: {
     334                                options: {
     335                                        configFile: 'node_modules/eslint-config-wordpress/index.js'
     336                                },
     337                                expand: true,
     338                                cwd: SOURCE_DIR + 'wp-content/themes',
     339                                src: [
     340                                        'twenty*/**/*.js',
     341                                        '!twenty{eleven,twelve,thirteen}/**',
     342                                        // Third party scripts
     343                                        '!twenty{fourteen,fifteen,sixteen,seventeen}/**/html5.js',
     344                                        '!twentyseventeen/assets/js/jquery.scrollTo.js'
     345                                ]
     346                        }
     347                },
    295348                jshint: {
    296349                        options: grunt.file.readJSON('.jshintrc'),
    297350                        grunt: {
    … … module.exports = function(grunt) { 
    736789
    737790                        if ( set.length ) {
    738791                                fs.stat( dir = set.shift(), function( error ) {
    739                                         error ? find( set ) : run( path.basename( dir ).substr( 1 ) );
     792                                        if ( error ) {
     793                                                find( set )
     794                                        } else {
     795                                                run( path.basename( dir ).substr( 1 ) );
     796                                        }
    740797                                } );
    741798                        } else {
    742799                                grunt.fatal( 'This WordPress install is not under version control.' );

  • package.json 

    diff --git a/package.json b/package.json
index 047d5fc35d..e4e419ac05 100644
    a b  
    1313  "license": "GPL-2.0+",
    1414  "devDependencies": {
    1515    "autoprefixer": "^6.5.1",
     16    "eslint": "~3.19.0",
     17    "eslint-config-wordpress": "~1.0.0",
    1618    "grunt": "~0.4.5",
    1719    "grunt-browserify": "~5.0.0",
    1820    "grunt-contrib-clean": "~1.0.0",
    … …  
    2527    "grunt-contrib-qunit": "^1.2.0",
    2628    "grunt-contrib-uglify": "~2.0.0",
    2729    "grunt-contrib-watch": "~1.0.0",
     30    "grunt-eslint": "~19.0.0",
    2831    "grunt-includes": "~0.5.1",
    2932    "grunt-jsvalidate": "~0.2.2",
    3033    "grunt-legacy-util": "^0.2.0",

  • src/wp-content/themes/twentyfourteen/js/slider.js 

    diff --git a/src/wp-content/themes/twentyfourteen/js/slider.js b/src/wp-content/themes/twentyfourteen/js/slider.js
index 1a08c6263a..d9fe3069ce 100644
    a b  
    2020                // Make variables public.
    2121                slider.vars = $.extend( {}, $.featuredslider.defaults, options );
    2222
    23                 namespace = slider.vars.namespace,
     23                namespace = slider.vars.namespace;
    2424
    2525                // Store a reference to the slider object.
    2626                $.data( el, 'featuredslider', slider );
    … …  
    105105                                        var type = 'control-paging',
    106106                                                j = 1,
    107107                                                item,
    108                                                 slide,
    109108                                                i;
    110109
    111110                                        slider.controlNavScaffold = $( '<ol class="' + namespace + 'control-nav ' + namespace + type + '"></ol>' );
    112111
    113112                                        if ( slider.pagingCount > 1 ) {
    114113                                                for ( i = 0; i < slider.pagingCount; i++ ) {
    115                                                         slide = slider.slides.eq( i );
    116114                                                        item = '<a>' + j + '</a>';
    117115                                                        slider.controlNavScaffold.append( '<li>' + item + '</li>' );
    118116                                                        j++;
    … …  
    120118                                        }
    121119
    122120                                        // CONTROLSCONTAINER
    123                                         ( slider.controlsContainer ) ? $( slider.controlsContainer ).append( slider.controlNavScaffold ) : slider.append( slider.controlNavScaffold );
     121                                        if ( slider.controlsContainer ) {
     122                                                $( slider.controlsContainer ).append( slider.controlNavScaffold );
     123                                        } else {
     124                                                slider.append( slider.controlNavScaffold );
     125                                        }
    124126                                        methods.controlNav.set();
    125127
    126128                                        methods.controlNav.active();
    … …  
    162164                                                slider.controlNav.eq( pos ).closest( 'li' ).remove();
    163165                                        }
    164166                                        methods.controlNav.set();
    165                                         ( slider.pagingCount > 1 && slider.pagingCount !== slider.controlNav.length ) ? slider.update( pos, action ) : methods.controlNav.active();
     167                                        if ( slider.pagingCount > 1 && slider.pagingCount !== slider.controlNav.length ) {
     168                                                slider.update( pos, action );
     169                                        } else {
     170                                                methods.controlNav.active();
     171                                        }
    166172                                }
    167173                        },
    168174
    … …  
    315321                                                return;
    316322                                        }
    317323
    318                                         transX = -e.translationX,
     324                                        transX = -e.translationX;
    319325                                        transY = -e.translationY;
    320326
    321327                                        // Accumulate translations.
    … …  
    349355                                        }
    350356
    351357                                        if ( slider.animatingTo === slider.currentSlide && ! scrolling && dx !== null ) {
    352                                                 updateDx = dx,
     358                                                updateDx = dx;
    353359                                                target = ( updateDx > 0 ) ? slider.getTarget( 'next' ) : slider.getTarget( 'prev' );
    354360
    355361                                                slider.featureAnimate( target );
    … …  
    376382
    377383                        smoothHeight: function( dur ) {
    378384                                var $obj = slider.viewport;
    379                                 ( dur ) ? $obj.animate( { 'height': slider.slides.eq( slider.animatingTo ).height() }, dur ) : $obj.height( slider.slides.eq( slider.animatingTo ).height() );
     385                                if ( dur ) {
     386                                        $obj.animate( { 'height': slider.slides.eq( slider.animatingTo ).height() }, dur );
     387                                } else {
     388                                        $obj.height( slider.slides.eq( slider.animatingTo ).height() );
     389                                }
    380390                        },
    381391
    382392                        setToClearWatchedEvent: function() {