From d83e8af56600172941a167f40ffa8372ad444872 Mon Sep 17 00:00:00 2001
From: Ricordisamoa <ricordisamoa@openmailbox.org>
Date: Tue, 13 Oct 2015 18:07:36 +0200
Subject: [PATCH] Use explicit methods instead of the jQuery constructor's
 second argument

As recommended by
https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Pitfalls:
"As of jQuery 1.4 the jQuery constructor has a new feature
 that allows passing an object as second argument, like
 jQuery( '<div>', { foo: 'bar', click: function () {}, css: { .. } } );.
 Don't use this. It makes code harder to follow, fails on attributes
 (such as 'size') that are also methods, and is unstable due to this
 mixing of jQuery methods with element attributes. A future jQuery
 method or plugin or called "title" might convert an element into a
 heading, which means the title attribute can also no longer be set
 through this method. Be explicit and call .attr(), .prop(), .on()
 etc. directly."

Change-Id: Iaf456b4b76dd4fc260dad0e4c0ec8f2976b59b83
---
 resources/src/jquery/jquery.spinner.js                    | 2 +-
 .../mediawiki.action/mediawiki.action.view.metadata.js    | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/resources/src/jquery/jquery.spinner.js b/resources/src/jquery/jquery.spinner.js
index 41c555b71c..af5a97d6a4 100644
--- a/resources/src/jquery/jquery.spinner.js
+++ b/resources/src/jquery/jquery.spinner.js
@@ -67,7 +67,7 @@
 
 			opts = $.extend( {}, defaults, opts );
 
-			var $spinner = $( '<div>', { 'class': 'mw-spinner', title: '...' } );
+			var $spinner = $( '<div>' ).addClass( 'mw-spinner' ).attr( 'title', '...' );
 			if ( opts.id !== undefined ) {
 				$spinner.attr( 'id', 'mw-spinner-' + opts.id );
 			}
diff --git a/resources/src/mediawiki.action/mediawiki.action.view.metadata.js b/resources/src/mediawiki.action/mediawiki.action.view.metadata.js
index 25b5acc5bc..b1a63b0506 100644
--- a/resources/src/mediawiki.action/mediawiki.action.view.metadata.js
+++ b/resources/src/mediawiki.action/mediawiki.action.view.metadata.js
@@ -21,10 +21,10 @@
 		$row = $( '<tr class="mw-metadata-show-hide-extended"></tr>' );
 		$col = $( '<td colspan="2"></td>' );
 
-		$link = $( '<a>', {
-			text: showText,
-			href: '#'
-		} ).click( function () {
+		$link = $( '<a>' )
+		.text( showText )
+		.attr( 'href', '#' )
+		.click( function () {
 			if ( $table.hasClass( 'collapsed' ) ) {
 				$( this ).text( hideText );
 			} else {
-- 
2.20.1