aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorScott González <scott.gonzalez@gmail.com>2012-10-25 06:19:25 -0400
committerScott González <scott.gonzalez@gmail.com>2012-10-25 06:19:25 -0400
commitbc44e61dd9e6797bb23bc3038d43436a8ba052c3 (patch)
treea5143b031ddfe169a60d7f4dc53b0f91b1311ee0
parent8d35c03dfdb3d4368bf38b113e25c06a82129cd5 (diff)
downloadjquery-ui-bc44e61dd9e6797bb23bc3038d43436a8ba052c3.tar.gz
jquery-ui-bc44e61dd9e6797bb23bc3038d43436a8ba052c3.zip
Accordion: Removed change and changestart events. Fixes #6841 - Accordion: Remove changestart event. Fixes #6843 - Accordion: Remove change event.
-rw-r--r--build/tasks/testswarm.js1
-rw-r--r--tests/unit/accordion/accordion.html3
-rw-r--r--tests/unit/accordion/accordion_common_deprecated.js22
-rw-r--r--tests/unit/accordion/accordion_deprecated.html140
-rw-r--r--tests/unit/accordion/accordion_deprecated.js99
-rw-r--r--tests/unit/all-active.html1
-rw-r--r--tests/unit/all.html1
-rw-r--r--tests/unit/subsuite.js1
-rw-r--r--ui/jquery.ui.accordion.js38
9 files changed, 0 insertions, 306 deletions
diff --git a/build/tasks/testswarm.js b/build/tasks/testswarm.js
index 7219bcf0c..96e33b0f4 100644
--- a/build/tasks/testswarm.js
+++ b/build/tasks/testswarm.js
@@ -10,7 +10,6 @@ var versions = {
},
tests = {
"Accordion": "accordion/accordion.html",
- "Accordion_deprecated": "accordion/accordion_deprecated.html",
"Autocomplete": "autocomplete/autocomplete.html",
"Button": "button/button.html",
"Core": "core/core.html",
diff --git a/tests/unit/accordion/accordion.html b/tests/unit/accordion/accordion.html
index b3b7caeae..4f40091bc 100644
--- a/tests/unit/accordion/accordion.html
+++ b/tests/unit/accordion/accordion.html
@@ -5,9 +5,6 @@
<title>jQuery UI Accordion Test Suite</title>
<script src="../../jquery.js"></script>
- <script>
- $.uiBackCompat = false;
- </script>
<link rel="stylesheet" href="../../../external/qunit.css">
<script src="../../../external/qunit.js"></script>
<script src="../../jquery.simulate.js"></script>
diff --git a/tests/unit/accordion/accordion_common_deprecated.js b/tests/unit/accordion/accordion_common_deprecated.js
deleted file mode 100644
index 3b1e0f4fe..000000000
--- a/tests/unit/accordion/accordion_common_deprecated.js
+++ /dev/null
@@ -1,22 +0,0 @@
-TestHelpers.commonWidgetTests( "accordion", {
- defaults: {
- active: 0,
- animate: {},
- collapsible: false,
- disabled: false,
- event: "click",
- header: "> li > :first-child,> :not(li):even",
- heightStyle: "auto",
- icons: {
- "activeHeader": "ui-icon-triangle-1-s",
- "header": "ui-icon-triangle-1-e"
- },
-
- // callbacks
- activate: null,
- beforeActivate: null,
- change: null,
- changestart: null,
- create: null
- }
-});
diff --git a/tests/unit/accordion/accordion_deprecated.html b/tests/unit/accordion/accordion_deprecated.html
deleted file mode 100644
index dfb9fa90e..000000000
--- a/tests/unit/accordion/accordion_deprecated.html
+++ /dev/null
@@ -1,140 +0,0 @@
-<!doctype html>
-<html lang="en">
-<head>
- <meta charset="utf-8">
- <title>jQuery UI Accordion Test Suite</title>
-
- <script src="../../jquery.js"></script>
- <link rel="stylesheet" href="../../../external/qunit.css">
- <script src="../../../external/qunit.js"></script>
- <script src="../../jquery.simulate.js"></script>
- <script src="../testsuite.js"></script>
- <script>
- TestHelpers.loadResources({
- css: [ "ui.core", "ui.accordion" ],
- js: [
- "ui/jquery.ui.core.js",
- "ui/jquery.ui.widget.js",
- "ui/jquery.ui.accordion.js"
- ]
- });
- </script>
-
- <script src="accordion_test_helpers.js"></script>
- <script src="accordion_common_deprecated.js"></script>
- <script src="accordion_core.js"></script>
- <script src="accordion_events.js"></script>
- <script src="accordion_methods.js"></script>
- <script src="accordion_options.js"></script>
- <script src="accordion_deprecated.js"></script>
-
- <script src="../swarminject.js"></script>
- <style>
- #list, #list1 *, #navigation, #navigation * {
- margin: 0;
- padding: 0;
- font-size: 12px;
- line-height: 15px;
- }
- </style>
-</head>
-<body>
-<h1 id="qunit-header">jQuery UI Accordion Test Suite</h1>
-<h2 id="qunit-banner"></h2>
-<div id="qunit-testrunner-toolbar"></div>
-<h2 id="qunit-userAgent"></h2>
-<ol id="qunit-tests"></ol>
-<div id="qunit-fixture">
-
-<div id="list1" class="foo">
- <h3 class="bar">There is one obvious advantage:</h3>
- <div class="foo">
- <p>
- You've seen it coming!
- <br>
- Buy now and get nothing for free!
- <br>
- Well, at least no free beer. Perhaps a bear, if you can afford it.
- </p>
- </div>
- <h3 class="bar">Now that you've got...</h3>
- <div class="foo">
- <p>
- your bear, you have to admit it!
- <br>
- No, we aren't <a href="#">selling bears</a>.
- </p>
- <p>
- We could talk about renting one.
- </p>
- </div>
- <h3 class="bar">Rent one bear, ...</h3>
- <div class="foo">
- <p>
- get two for three beer.
- </p>
- <p>
- And now, for something completely different.
- </p>
- </div>
-</div>
-
-<div id="navigationWrapper">
- <ul id="navigation">
- <li>
- <h2><a href="?p=1.1.1">Guitar</a></h2>
- <ul>
- <li><a href="?p=1.1.1.1">Electric</a></li>
- <li><a href="?p=1.1.1.2">Acoustic</a></li>
- <li><a href="?p=1.1.1.3">Amps</a></li>
- <li><a href="?p=1.1.1.4">Effects</a></li>
- <li><a href="?p=1.1.1.5">Accessories</a></li>
- </ul>
- </li>
- <li>
- <h2><a href="?p=1.1.2"><span>Bass</span></a></h2>
- <ul>
- <li><a href="?p=1.1.2.1">Electric</a></li>
- <li><a href="?p=1.1.2.2">Acoustic</a></li>
- <li><a href="?p=1.1.2.3">Amps</a></li>
- <li><a href="?p=1.1.2.4">Effects</a></li>
- <li><a href="?p=1.1.2.5">Accessories</a></li>
- <li><a href="?p=1.1.2.5">Accessories</a></li>
- <li><a href="?p=1.1.2.5">Accessories</a></li>
- </ul>
- </li>
- <li>
- <h2><a href="?p=1.1.3">Drums</a></h2>
- <ul>
- <li><a href="?p=1.1.3.2">Acoustic</a></li>
- <li><a href="?p=1.1.3.3">Electronic</a></li>
- <li><a href="?p=1.1.3.6">Accessories</a></li>
- </ul>
- </li>
- </ul>
-</div>
-
-<dl id="accordion-dl">
- <dt>
- Accordion Header 1
- </dt>
- <dd>
- Accordion Content 1
- </dd>
- <dt>
- Accordion Header 2
- </dt>
- <dd>
- Accordion Content 2
- </dd>
- <dt>
- Accordion Header 3
- </dt>
- <dd>
- Accordion Content 3
- </dd>
-</dl>
-
-</div>
-</body>
-</html>
diff --git a/tests/unit/accordion/accordion_deprecated.js b/tests/unit/accordion/accordion_deprecated.js
deleted file mode 100644
index 57dabf418..000000000
--- a/tests/unit/accordion/accordion_deprecated.js
+++ /dev/null
@@ -1,99 +0,0 @@
-(function( $ ) {
-
-var equalHeight = TestHelpers.accordion.equalHeight,
- setupTeardown = TestHelpers.accordion.setupTeardown,
- state = TestHelpers.accordion.state;
-
-module( "accordion (deprecated) - changestart/change events", setupTeardown() );
-
-test( "changestart", function() {
- expect( 26 );
- var element = $( "#list1" ).accordion({
- active: false,
- collapsible: true
- }),
- headers = element.find( ".ui-accordion-header" ),
- content = element.find( ".ui-accordion-content" );
-
- element.one( "accordionchangestart", function( event, ui ) {
- equal( ui.oldHeader.length, 0 );
- equal( ui.oldContent.length, 0 );
- equal( ui.newHeader.length, 1 );
- strictEqual( ui.newHeader[ 0 ], headers[ 0 ] );
- equal( ui.newContent.length, 1 );
- strictEqual( ui.newContent[ 0 ], content[ 0 ] );
- state( element, 0, 0, 0 );
- });
- element.accordion( "option", "active", 0 );
- state( element, 1, 0, 0 );
-
- element.one( "accordionchangestart", function( event, ui ) {
- equal( ui.oldHeader.length, 1 );
- strictEqual( ui.oldHeader[ 0 ], headers[ 0 ] );
- equal( ui.oldContent.length, 1 );
- strictEqual( ui.oldContent[ 0 ], content[ 0 ] );
- equal( ui.newHeader.length, 1 );
- strictEqual( ui.newHeader[ 0 ], headers[ 1 ] );
- equal( ui.newContent.length, 1 );
- strictEqual( ui.newContent[ 0 ], content[ 1 ] );
- state( element, 1, 0, 0 );
- });
- headers.eq( 1 ).click();
- state( element, 0, 1, 0 );
-
- element.one( "accordionchangestart", function( event, ui ) {
- equal( ui.oldHeader.length, 1 );
- strictEqual( ui.oldHeader[ 0 ], headers[ 1 ] );
- equal( ui.oldContent.length, 1 );
- strictEqual( ui.oldContent[ 0 ], content[ 1 ] );
- equal( ui.newHeader.length, 0 );
- equal( ui.newContent.length, 0 );
- state( element, 0, 1, 0 );
- });
- element.accordion( "option", "active", false );
- state( element, 0, 0, 0 );
-});
-
-test( "change", function() {
- expect( 20 );
- var element = $( "#list1" ).accordion({
- active: false,
- collapsible: true
- }),
- headers = element.find( ".ui-accordion-header" ),
- content = element.find( ".ui-accordion-content" );
-
- element.one( "accordionchange", function( event, ui ) {
- equal( ui.oldHeader.length, 0 );
- equal( ui.oldContent.length, 0 );
- equal( ui.newHeader.length, 1 );
- strictEqual( ui.newHeader[ 0 ], headers[ 0 ] );
- equal( ui.newContent.length, 1 );
- strictEqual( ui.newContent[ 0 ], content[ 0 ] );
- });
- element.accordion( "option", "active", 0 );
-
- element.one( "accordionchange", function( event, ui ) {
- equal( ui.oldHeader.length, 1 );
- strictEqual( ui.oldHeader[ 0 ], headers[ 0 ] );
- equal( ui.oldContent.length, 1 );
- strictEqual( ui.oldContent[ 0 ], content[ 0 ] );
- equal( ui.newHeader.length, 1 );
- strictEqual( ui.newHeader[ 0 ], headers[ 1 ] );
- equal( ui.newContent.length, 1 );
- strictEqual( ui.newContent[ 0 ], content[ 1 ] );
- });
- headers.eq( 1 ).click();
-
- element.one( "accordionchange", function( event, ui ) {
- equal( ui.oldHeader.length, 1 );
- strictEqual( ui.oldHeader[ 0 ], headers[ 1 ] );
- equal( ui.oldContent.length, 1 );
- strictEqual( ui.oldContent[ 0 ], content[ 1 ] );
- equal( ui.newHeader.length, 0 );
- equal( ui.newContent.length, 0 );
- });
- element.accordion( "option", "active", false );
-});
-
-})(jQuery);
diff --git a/tests/unit/all-active.html b/tests/unit/all-active.html
index f842aa5cc..236411475 100644
--- a/tests/unit/all-active.html
+++ b/tests/unit/all-active.html
@@ -17,7 +17,6 @@
var params = [],
suites = [
"accordion/accordion.html",
- "accordion/accordion_deprecated.html",
"autocomplete/autocomplete.html",
"button/button.html",
"core/core.html",
diff --git a/tests/unit/all.html b/tests/unit/all.html
index bc204c976..e8dac887c 100644
--- a/tests/unit/all.html
+++ b/tests/unit/all.html
@@ -17,7 +17,6 @@
var params = [],
suites = [
"accordion/accordion.html",
- "accordion/accordion_deprecated.html",
"autocomplete/autocomplete.html",
"button/button.html",
"core/core.html",
diff --git a/tests/unit/subsuite.js b/tests/unit/subsuite.js
index 6238cb2b1..b583bbd75 100644
--- a/tests/unit/subsuite.js
+++ b/tests/unit/subsuite.js
@@ -7,7 +7,6 @@ var versions = [
"git"
],
additionalTests = {
- accordion: [ "accordion_deprecated.html" ],
position: [ "position_deprecated.html" ]
};
diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js
index 6ec85b681..b22f0c4e2 100644
--- a/ui/jquery.ui.accordion.js
+++ b/ui/jquery.ui.accordion.js
@@ -540,42 +540,4 @@ $.widget( "ui.accordion", {
}
});
-
-
-// DEPRECATED
-if ( $.uiBackCompat !== false ) {
- // change events
- (function( $, prototype ) {
- $.extend( prototype.options, {
- change: null,
- changestart: null
- });
-
- var _trigger = prototype._trigger;
- prototype._trigger = function( type, event, data ) {
- var ret = _trigger.apply( this, arguments );
- if ( !ret ) {
- return false;
- }
-
- if ( type === "beforeActivate" ) {
- ret = _trigger.call( this, "changestart", event, {
- oldHeader: data.oldHeader,
- oldContent: data.oldPanel,
- newHeader: data.newHeader,
- newContent: data.newPanel
- });
- } else if ( type === "activate" ) {
- ret = _trigger.call( this, "change", event, {
- oldHeader: data.oldHeader,
- oldContent: data.oldPanel,
- newHeader: data.newHeader,
- newContent: data.newPanel
- });
- }
- return ret;
- };
- }( jQuery, jQuery.ui.accordion.prototype ) );
-}
-
})( jQuery );