From 235e92846eb6fd132f28ec5e05240e3821542ac8 Mon Sep 17 00:00:00 2001 From: "James D. Forrester" Date: Wed, 25 Oct 2017 12:38:18 -0700 Subject: [PATCH] resources: Remove deprecated 'jquery.placeholder' module Change-Id: I1f14561e54df477067460411ff155e3f2eeb167a --- RELEASE-NOTES-1.31 | 1 + resources/Resources.php | 6 ------ resources/src/jquery/jquery.placeholder.js | 18 ------------------ .../src/mediawiki.less/mediawiki.mixins.less | 4 ---- 4 files changed, 1 insertion(+), 28 deletions(-) delete mode 100644 resources/src/jquery/jquery.placeholder.js diff --git a/RELEASE-NOTES-1.31 b/RELEASE-NOTES-1.31 index 17dba874d2..5e2df5ff27 100644 --- 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. +* The deprecated 'jquery.placeholder' module was removed. * … === Bug fixes in 1.31 === diff --git a/resources/Resources.php b/resources/Resources.php index 545a84c058..9acc0ef810 100644 --- a/resources/Resources.php +++ b/resources/Resources.php @@ -286,12 +286,6 @@ return [ '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', diff --git a/resources/src/jquery/jquery.placeholder.js b/resources/src/jquery/jquery.placeholder.js deleted file mode 100644 index afb8837ff7..0000000000 --- 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 fb3b00ed8c..ea0b95993d 100644 --- a/resources/src/mediawiki.less/mediawiki.mixins.less +++ b/resources/src/mediawiki.less/mediawiki.mixins.less @@ -171,10 +171,6 @@ &::placeholder { @rules(); } - // For inputs that use jquery.placeholder.js e.g. IE9 - &.placeholder { - @rules(); - } } /* stylelint-enable selector-no-vendor-prefix, at-rule-no-unknown */ -- 2.20.1