CSS: Include `show`, `hide` & `toggle` methods in the jQuery slim build

The `show()`, `hide()` & `toggle()` methods were included in the 3.x jQuery
slim build. The jQuery master build accidentally started to exclude them as
they were only imported in the effects module and the new Rollup-based build
system follows the module dependency graph when excluding modules.

To resolve the issue, import the `css/showHide.js` file directly in the main
`jquery.js` file.

Closes gh-4704
Ref jquery/jquery-migrate#346

diff --git a/src/jquery.js b/src/jquery.js
index 3a27fe3ff94ce6e0b34fe1dc27a6b9486ea5fca6..d9fd61afaced33e06bdbf49971bc1389d35b218a 100644 (file)
--- a/src/jquery.js
+++ b/src/jquery.js
@@ -16,6 +16,7 @@ import "./manipulation/_evalUrl.js";
 import "./wrap.js";
 import "./css.js";
 import "./css/hiddenVisibleSelectors.js";
+import "./css/showHide.js";
 import "./serialize.js";
 import "./ajax.js";
 import "./ajax/xhr.js";

diff --git a/test/unit/css.js b/test/unit/css.js
index 54a9a2fcc92d221e28c9796300f9ebf6520c33fe..095f3641bac35c72de63d40abd726dd29e9cac42 100644 (file)
--- a/test/unit/css.js
+++ b/test/unit/css.js
@@ -487,9 +487,6 @@ QUnit.test( "css(Object) where values are Functions with incoming values", funct
        jQuery( "#cssFunctionTest" ).remove();
 } );
 
-// .show(), .hide(), can be excluded from the build
-if ( jQuery.fn.show && jQuery.fn.hide ) {
-
 QUnit.test( "show()", function( assert ) {
 
        assert.expect( 18 );
@@ -968,8 +965,6 @@ QUnit.test( "show/hide 3.0, inline hidden", function( assert ) {
        } );
 } );
 
-}
-
 QUnit[ QUnit.jQuerySelectors && jQuery.fn.toggle ? "test" : "skip" ]( "toggle()", function( assert ) {
        assert.expect( 9 );
        var div, oldHide,