resources: Remove deprecated 'jquery.placeholder' module

Change-Id: I1f14561e54df477067460411ff155e3f2eeb167a

diff --git a/RELEASE-NOTES-1.31 b/RELEASE-NOTES-1.31
index 17dba87..5e2df5f 100644 (file)
--- a/RELEASE-NOTES-1.31
+++ b/RELEASE-NOTES-1.31
@@ -33,6 +33,7 @@ production.
 * (T17845) The deprecated 'jquery.badge' module was removed.
 * The deprecated 'jquery.autoEllipsis' module was removed. Use the CSS
   text-overflow property instead.
 * (T17845) The deprecated 'jquery.badge' module was removed.
 * The deprecated 'jquery.autoEllipsis' module was removed. Use the CSS
   text-overflow property instead.

+* The deprecated 'jquery.placeholder' module was removed.

 * …
 
 === Bug fixes in 1.31 ===
 * …
 
 === Bug fixes in 1.31 ===

diff --git a/resources/Resources.php b/resources/Resources.php
index 545a84c..9acc0ef 100644 (file)
--- a/resources/Resources.php
+++ b/resources/Resources.php
@@ -286,12 +286,6 @@ return [
                'scripts' => 'resources/src/jquery/jquery.mw-jump.js',
                'targets' => [ 'desktop', 'mobile' ],
        ],
                'scripts' => 'resources/src/jquery/jquery.mw-jump.js',
                'targets' => [ 'desktop', 'mobile' ],
        ],

-       'jquery.placeholder' => [
-               'deprecated' => 'Use of "jquery.placeholder" is deprecated since MediaWiki 1.29.0',
-
-               'scripts' => 'resources/src/jquery/jquery.placeholder.js',
-               'targets' => [ 'desktop', 'mobile' ],
-       ],

        'jquery.qunit' => [
                'scripts' => 'resources/lib/qunitjs/qunit.js',
                'styles' => 'resources/lib/qunitjs/qunit.css',
        'jquery.qunit' => [
                'scripts' => 'resources/lib/qunitjs/qunit.js',
                'styles' => 'resources/lib/qunitjs/qunit.css',

diff --git a/resources/src/jquery/jquery.placeholder.js b/resources/src/jquery/jquery.placeholder.js
deleted file mode 100644 (file)
index afb8837..0000000
--- a/resources/src/jquery/jquery.placeholder.js
+++ /dev/null
@@ -1,18 +0,0 @@
-/*!
- * No-op for compatibility with code from before we used
- * native placeholder in all supported browsers.
- */
-
-( function ( $ ) {
-       var placeholder;
-
-       placeholder = $.fn.placeholder = function ( text ) {
-               if ( arguments.length ) {
-                       this.prop( 'placeholder', text );
-               }
-               return this;
-       };
-
-       placeholder.input = placeholder.textarea = true;
-
-}( jQuery ) );

diff --git a/resources/src/mediawiki.less/mediawiki.mixins.less b/resources/src/mediawiki.less/mediawiki.mixins.less
index fb3b00e..ea0b959 100644 (file)
--- a/resources/src/mediawiki.less/mediawiki.mixins.less
+++ b/resources/src/mediawiki.less/mediawiki.mixins.less
@@ -171,10 +171,6 @@
        &::placeholder {
                @rules();
        }
        &::placeholder {
                @rules();
        }

-       // For inputs that use jquery.placeholder.js e.g. IE9
-       &.placeholder {
-               @rules();
-       }

 }
 /* stylelint-enable selector-no-vendor-prefix, at-rule-no-unknown */
 
 }
 /* stylelint-enable selector-no-vendor-prefix, at-rule-no-unknown */