aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFelix Nagel <info@felixnagel.com>2013-04-21 15:10:38 +0200
committerFelix Nagel <info@felixnagel.com>2013-04-21 15:10:38 +0200
commit4efd006e26a6cdc3bf78832c0a0832e7301c841b (patch)
tree5263fe72e9a2784410632d2dee56efd4e58b83f2
parent73c7342bc4418a902bf021c89e7d2c43172e004e (diff)
parent71a332e8b83a1657521e04388f5592997e81bbcc (diff)
downloadjquery-ui-4efd006e26a6cdc3bf78832c0a0832e7301c841b.tar.gz
jquery-ui-4efd006e26a6cdc3bf78832c0a0832e7301c841b.zip
Merge branch 'master' into selectmenu
-rw-r--r--.jshintrc11
-rw-r--r--.mailmap2
-rw-r--r--Gruntfile.js4
-rw-r--r--build/release/release.js2
-rw-r--r--build/tasks/build.js2
-rw-r--r--build/tasks/testswarm.js45
-rw-r--r--demos/autocomplete/combobox.html14
-rw-r--r--package.json2
-rw-r--r--tests/.jshintrc16
-rw-r--r--tests/jquery.simulate.js8
-rw-r--r--tests/unit/button/button.html2
-rw-r--r--tests/unit/button/button_core.js37
-rw-r--r--tests/unit/datepicker/datepicker_core.js373
-rw-r--r--tests/unit/datepicker/datepicker_options.js238
-rw-r--r--tests/unit/datepicker/datepicker_test_helpers.js21
-rw-r--r--tests/unit/draggable/draggable_core.js85
-rw-r--r--tests/unit/draggable/draggable_options.js44
-rw-r--r--tests/unit/draggable/draggable_test_helpers.js61
-rw-r--r--ui/.jshintrc12
-rw-r--r--ui/i18n/jquery.ui.datepicker-sr-SR.js2
-rw-r--r--ui/i18n/jquery.ui.datepicker-sr.js2
-rw-r--r--ui/jquery.ui.button.js31
-rw-r--r--ui/jquery.ui.core.js9
-rw-r--r--ui/jquery.ui.datepicker.js42
-rw-r--r--ui/jquery.ui.draggable.js101
25 files changed, 723 insertions, 443 deletions
diff --git a/.jshintrc b/.jshintrc
index d6966c51d..d34c42da5 100644
--- a/.jshintrc
+++ b/.jshintrc
@@ -1,12 +1,17 @@
{
+ "boss": true,
"curly": true,
- "eqnull": true,
"eqeqeq": true,
+ "eqnull": true,
"expr": true,
+ "immed": true,
"noarg": true,
- "node": true,
"onevar": true,
+ "quotmark": "double",
+ "smarttabs": true,
"trailing": true,
"undef": true,
- "unused": true
+ "unused": true,
+
+ "node": true
}
diff --git a/.mailmap b/.mailmap
index f83fbe970..4158c202c 100644
--- a/.mailmap
+++ b/.mailmap
@@ -11,7 +11,7 @@ Ben Hollis <bhollis@amazon.com> <ben@benhollis.net>
Benjamin Scott Boyle <benjamins.boyle@gmail.com>
Bert ter Heide <bertjh@hotmail.com>
Chairat Sunthornwiphat <pipo@sixhead.com>
-Corey Frang <gnarf@gnarf.net>
+Corey Frang <gnarf37@gmail.com> <gnarf@gnarf.net>
Courtland Allen <courtlandallen@gmail.com>
Dan Streetman <ddstreet@ieee.org>
Diego Tres <diegotres@gmail.com>
diff --git a/Gruntfile.js b/Gruntfile.js
index f00343d58..e93df3238 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -74,7 +74,7 @@ var
},
main: {
options: {
- keepSpecialComments: '*'
+ keepSpecialComments: "*"
},
src: "dist/jquery-ui.css",
dest: "dist/jquery-ui.min.css"
@@ -319,7 +319,7 @@ grunt.initConfig({
files: expandFiles( "tests/unit/**/*.html" ).filter(function( file ) {
// disabling everything that doesn't (quite) work with PhantomJS for now
// TODO except for all|index|test, try to include more as we go
- return !( /(all|index|test|dialog|tabs|tooltip)\.html$/ ).test( file );
+ return !( /(all|index|test|dialog|tooltip)\.html$/ ).test( file );
})
},
jshint: {
diff --git a/build/release/release.js b/build/release/release.js
index 2eab5283f..159345b99 100644
--- a/build/release/release.js
+++ b/build/release/release.js
@@ -1,5 +1,5 @@
#!/usr/bin/env node
-/*global cat:true cd:true echo:true exec:true exit:true*/
+/* global cat:true, cd:true, echo:true, exec:true, exit:true */
// Usage:
// stable release: node release.js
diff --git a/build/tasks/build.js b/build/tasks/build.js
index 978bee004..09243cfb6 100644
--- a/build/tasks/build.js
+++ b/build/tasks/build.js
@@ -140,7 +140,7 @@ grunt.registerMultiTask( "zip", "Create a zip file for release", function() {
cmd: "zip",
args: [ "-r", dest, this.data.src ],
opts: {
- cwd: 'dist'
+ cwd: "dist"
}
}, function( err ) {
if ( err ) {
diff --git a/build/tasks/testswarm.js b/build/tasks/testswarm.js
index 3166e9277..825b23b0c 100644
--- a/build/tasks/testswarm.js
+++ b/build/tasks/testswarm.js
@@ -34,37 +34,40 @@ var versions = {
"Widget": "widget/widget.html"
};
-function submit( commit, runs, configFile, version, done ) {
+function submit( commit, runs, configFile, extra, done ) {
var testName,
testswarm = require( "testswarm" ),
- config = grunt.file.readJSON( configFile ).jqueryui;
- version = version ? ( version + " " ) : "";
+ config = grunt.file.readJSON( configFile ).jqueryui,
+ commitUrl = "https://github.com/jquery/jquery-ui/commit/" + commit;
+
+ if ( extra ) {
+ extra = " " + extra;
+ }
+
for ( testName in runs ) {
- runs[ testName] = config.testUrl + commit + "/tests/unit/" + runs[ testName ];
+ runs[ testName ] = config.testUrl + commit + "/tests/unit/" + runs[ testName ];
}
- testswarm.createClient( {
+ testswarm.createClient({
url: config.swarmUrl,
pollInterval: 10000,
timeout: 1000 * 60 * 45
- } )
+ })
.addReporter( testswarm.reporters.cli )
- .auth( {
+ .auth({
id: config.authUsername,
token: config.authToken
- } )
- .addjob(
- {
- name: 'jQuery UI ' + version + '#<a href="https://github.com/jquery/jquery-ui/commit/' + commit + '">' + commit.substr( 0, 10 ) + '</a>',
- runs: runs,
- runMax: config.runMax,
- browserSets: config.browserSets
- }, function( err, passed ) {
- if ( err ) {
- grunt.log.error( err );
- }
- done( passed );
+ })
+ .addjob({
+ name: "jQuery UI #<a href='" + commitUrl + "'>" + commit.substr( 0, 10 ) + "</a>" + extra,
+ runs: runs,
+ runMax: config.runMax,
+ browserSets: config.browserSets
+ }, function( error, passed ) {
+ if ( error ) {
+ grunt.log.error( error );
}
- );
+ done( passed );
+ });
}
grunt.registerTask( "testswarm", function( commit, configFile ) {
@@ -83,7 +86,7 @@ grunt.registerTask( "testswarm-multi-jquery", function( commit, configFile, mino
allTests[ test + "-" + version ] = tests[ test ] + "?nojshint=true&jquery=" + version;
}
});
- submit( commit, allTests, configFile, minor + " core", this.async() );
+ submit( commit, allTests, configFile, "core " + minor, this.async() );
});
};
diff --git a/demos/autocomplete/combobox.html b/demos/autocomplete/combobox.html
index 22f34aa17..2b5e511be 100644
--- a/demos/autocomplete/combobox.html
+++ b/demos/autocomplete/combobox.html
@@ -14,11 +14,11 @@
<script src="../../ui/jquery.ui.tooltip.js"></script>
<link rel="stylesheet" href="../demos.css">
<style>
- .ui-combobox {
+ .custom-combobox {
position: relative;
display: inline-block;
}
- .ui-combobox-toggle {
+ .custom-combobox-toggle {
position: absolute;
top: 0;
bottom: 0;
@@ -28,17 +28,17 @@
*height: 1.7em;
*top: 0.1em;
}
- .ui-combobox-input {
+ .custom-combobox-input {
margin: 0;
padding: 0.3em;
}
</style>
<script>
(function( $ ) {
- $.widget( "ui.combobox", {
+ $.widget( "custom.combobox", {
_create: function() {
this.wrapper = $( "<span>" )
- .addClass( "ui-combobox" )
+ .addClass( "custom-combobox" )
.insertAfter( this.element );
this.element.hide();
@@ -54,7 +54,7 @@
.appendTo( this.wrapper )
.val( value )
.attr( "title", "" )
- .addClass( "ui-state-default ui-combobox-input ui-widget ui-widget-content ui-corner-left" )
+ .addClass( "custom-combobox-input ui-widget ui-widget-content ui-state-default ui-corner-left" )
.autocomplete({
delay: 0,
minLength: 0,
@@ -92,7 +92,7 @@
text: false
})
.removeClass( "ui-corner-all" )
- .addClass( "ui-corner-right ui-combobox-toggle" )
+ .addClass( "custom-combobox-toggle ui-corner-right" )
.mousedown(function() {
wasOpen = input.autocomplete( "widget" ).is( ":visible" );
})
diff --git a/package.json b/package.json
index f348f88cd..4a66f0d73 100644
--- a/package.json
+++ b/package.json
@@ -54,7 +54,7 @@
"dependencies": {},
"devDependencies": {
"grunt": "0.4.1",
- "grunt-contrib-jshint": "0.1.1",
+ "grunt-contrib-jshint": "0.4.1",
"grunt-contrib-uglify": "0.1.1",
"grunt-contrib-concat": "0.1.3",
"grunt-contrib-qunit": "0.2.0",
diff --git a/tests/.jshintrc b/tests/.jshintrc
index b987c5df6..26f47fcee 100644
--- a/tests/.jshintrc
+++ b/tests/.jshintrc
@@ -1,18 +1,22 @@
{
- "browser": true,
+ "boss": true,
"curly": true,
- "eqnull": true,
"eqeqeq": true,
+ "eqnull": true,
"expr": true,
- "evil": true,
- "jquery": true,
- "latedef": true,
+ "immed": true,
"noarg": true,
"onevar": true,
"quotmark": "double",
+ "smarttabs": true,
"trailing": true,
"undef": true,
"unused": true,
+
+ "browser": true,
+ "evil": true,
+ "jquery": true,
+
"globals": {
"asyncTest": false,
"closeEnough": false,
@@ -34,4 +38,4 @@
"TestHelpers": true,
"JSHINT": false
}
-} \ No newline at end of file
+}
diff --git a/tests/jquery.simulate.js b/tests/jquery.simulate.js
index 6e2f3ba8e..0a0c42487 100644
--- a/tests/jquery.simulate.js
+++ b/tests/jquery.simulate.js
@@ -316,8 +316,12 @@ $.extend( $.simulate.prototype, {
this.simulateEvent( document, "mousemove", coord );
}
- this.simulateEvent( target, "mouseup", coord );
- this.simulateEvent( target, "click", coord );
+ if ( $.contains( document, target ) ) {
+ this.simulateEvent( target, "mouseup", coord );
+ this.simulateEvent( target, "click", coord );
+ } else {
+ this.simulateEvent( document, "mouseup", coord );
+ }
}
});
diff --git a/tests/unit/button/button.html b/tests/unit/button/button.html
index eeb568686..223581ef7 100644
--- a/tests/unit/button/button.html
+++ b/tests/unit/button/button.html
@@ -71,6 +71,8 @@
<div><input id="submit" type="submit" value="Label"></div>
+<button id="button1">Button</button>
+
</div>
</body>
</html>
diff --git a/tests/unit/button/button_core.js b/tests/unit/button/button_core.js
index dbc079818..6df7da9db 100644
--- a/tests/unit/button/button_core.js
+++ b/tests/unit/button/button_core.js
@@ -153,6 +153,30 @@ test( "#6262 - buttonset not applying ui-corner to invisible elements", function
ok( set.find( "label:eq(2)" ).is( ".ui-button.ui-corner-right" ) );
});
+asyncTest( "Resetting a button's form should refresh the visual state of the button widget to match.", function() {
+ expect( 2 );
+ var form = $( "<form>" +
+ "<button></button>" +
+ "<label for='c1'></label><input id='c1' type='checkbox' checked>" +
+ "</form>" ),
+ button = form.find( "button" ).button(),
+ checkbox = form.find( "input[type=checkbox]" ).button();
+
+ checkbox.prop( "checked", false ).button( "refresh" );
+ ok( !checkbox.button( "widget" ).hasClass( "ui-state-active" ) );
+
+ form.get( 0 ).reset();
+
+ // #9213: If a button has been removed, refresh should not be called on it when
+ // its corresponding form is reset.
+ button.remove();
+
+ setTimeout(function() {
+ ok( checkbox.button( "widget" ).hasClass( "ui-state-active" ));
+ start();
+ });
+});
+
asyncTest( "#6711 Checkbox/Radiobutton do not Show Focused State when using Keyboard Navigation", function() {
expect( 2 );
var check = $( "#check" ).button(),
@@ -172,4 +196,17 @@ test( "#7534 - Button label selector works for ids with \":\"", function() {
ok( group.find( "label" ).is( ".ui-button" ), "Found an id with a :" );
});
+asyncTest( "#9169 - Disabled button maintains ui-state-focus", function() {
+ expect( 2 );
+ var element = $( "#button1" ).button();
+ element[ 0 ].focus();
+ setTimeout(function() {
+ ok( element.hasClass( "ui-state-focus" ), "button has ui-state-focus" );
+ element.button( "disable" );
+ ok( !element.hasClass( "ui-state-focus" ),
+ "button does not have ui-state-focus when disabled" );
+ start();
+ });
+});
+
})(jQuery);
diff --git a/tests/unit/datepicker/datepicker_core.js b/tests/unit/datepicker/datepicker_core.js
index befda3b2a..0bc5723ec 100644
--- a/tests/unit/datepicker/datepicker_core.js
+++ b/tests/unit/datepicker/datepicker_core.js
@@ -30,154 +30,173 @@ test("widget method", function() {
deepEqual($("body > #ui-datepicker-div:last-child")[0], actual);
});
-asyncTest("baseStructure", function() {
+asyncTest( "baseStructure", function() {
expect( 58 );
var header, title, table, thead, week, panel, inl, child,
- inp = TestHelpers.datepicker.init("#inp"),
- dp = $("#ui-datepicker-div");
+ inp = TestHelpers.datepicker.initNewInput(),
+ dp = $( "#ui-datepicker-div" );
function step1() {
- inp[0].focus();
- setTimeout(function() {
- ok(dp.is(":visible"), "Structure - datepicker visible");
- ok(!dp.is(".ui-datepicker-rtl"), "Structure - not right-to-left");
- ok(!dp.is(".ui-datepicker-multi"), "Structure - not multi-month");
- equal(dp.children().length, 2, "Structure - child count");
-
- header = dp.children(":first");
- ok(header.is("div.ui-datepicker-header"), "Structure - header division");
- equal(header.children().length, 3, "Structure - header child count");
- ok(header.children(":first").is("a.ui-datepicker-prev") && header.children(":first").html() !== "", "Structure - prev link");
- ok(header.children(":eq(1)").is("a.ui-datepicker-next") && header.children(":eq(1)").html() !== "", "Structure - next link");
-
- title = header.children(":last");
- ok(title.is("div.ui-datepicker-title") && title.html() !== "","Structure - title division");
- equal(title.children().length, 2, "Structure - title child count");
- ok(title.children(":first").is("span.ui-datepicker-month") && title.children(":first").text() !== "", "Structure - month text");
- ok(title.children(":last").is("span.ui-datepicker-year") && title.children(":last").text() !== "", "Structure - year text");
-
- table = dp.children(":eq(1)");
- ok(table.is("table.ui-datepicker-calendar"), "Structure - month table");
- ok(table.children(":first").is("thead"), "Structure - month table thead");
- thead = table.children(":first").children(":first");
- ok(thead.is("tr"), "Structure - month table title row");
- equal(thead.find("th").length, 7, "Structure - month table title cells");
- ok(table.children(":eq(1)").is("tbody"), "Structure - month table body");
- ok(table.children(":eq(1)").children("tr").length >= 4, "Structure - month table week count");
- week = table.children(":eq(1)").children(":first");
- ok(week.is("tr"), "Structure - month table week row");
- equal(week.children().length, 7, "Structure - week child count");
- ok(week.children(":first").is("td.ui-datepicker-week-end"), "Structure - month table first day cell");
- ok(week.children(":last").is("td.ui-datepicker-week-end"), "Structure - month table second day cell");
- inp.datepicker("hide").datepicker("destroy");
-
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ":visible" ), "Structure - datepicker visible" );
+ ok( !dp.is( ".ui-datepicker-rtl" ), "Structure - not right-to-left" );
+ ok( !dp.is( ".ui-datepicker-multi" ), "Structure - not multi-month" );
+ equal( dp.children().length, 2, "Structure - child count" );
+
+ header = dp.children( ":first" );
+ ok( header.is( "div.ui-datepicker-header" ), "Structure - header division" );
+ equal( header.children().length, 3, "Structure - header child count" );
+ ok( header.children( ":first" ).is( "a.ui-datepicker-prev" ) && header.children( ":first" ).html() !== "", "Structure - prev link" );
+ ok( header.children( ":eq(1)" ).is( "a.ui-datepicker-next" ) && header.children( ":eq(1)" ).html() !== "", "Structure - next link" );
+
+ title = header.children( ":last" );
+ ok( title.is( "div.ui-datepicker-title" ) && title.html() !== "","Structure - title division" );
+ equal( title.children().length, 2, "Structure - title child count" );
+ ok( title.children( ":first" ).is( "span.ui-datepicker-month" ) && title.children( ":first" ).text() !== "", "Structure - month text" );
+ ok( title.children( ":last" ).is( "span.ui-datepicker-year" ) && title.children( ":last" ).text() !== "", "Structure - year text" );
+
+ table = dp.children( ":eq(1)" );
+ ok( table.is( "table.ui-datepicker-calendar" ), "Structure - month table" );
+ ok( table.children( ":first" ).is( "thead" ), "Structure - month table thead" );
+
+ thead = table.children( ":first" ).children( ":first" );
+ ok( thead.is( "tr" ), "Structure - month table title row" );
+ equal( thead.find( "th" ).length, 7, "Structure - month table title cells" );
+ ok( table.children( ":eq(1)" ).is( "tbody" ), "Structure - month table body" );
+ ok( table.children( ":eq(1)" ).children( "tr" ).length >= 4, "Structure - month table week count" );
+
+ week = table.children( ":eq(1)" ).children( ":first" );
+ ok( week.is( "tr" ), "Structure - month table week row" );
+ equal( week.children().length, 7, "Structure - week child count" );
+ ok( week.children( ":first" ).is( "td.ui-datepicker-week-end" ), "Structure - month table first day cell" );
+ ok( week.children( ":last" ).is( "td.ui-datepicker-week-end" ), "Structure - month table second day cell" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step2();
});
}
function step2() {
// Editable month/year and button panel
- inp = TestHelpers.datepicker.init("#inp", {changeMonth: true, changeYear: true, showButtonPanel: true});
- inp.focus();
- setTimeout(function() {
- title = dp.find("div.ui-datepicker-title");
- ok(title.children(":first").is("select.ui-datepicker-month"), "Structure - month selector");
- ok(title.children(":last").is("select.ui-datepicker-year"), "Structure - year selector");
-
- panel = dp.children(":last");
- ok(panel.is("div.ui-datepicker-buttonpane"), "Structure - button panel division");
- equal(panel.children().length, 2, "Structure - button panel child count");
- ok(panel.children(":first").is("button.ui-datepicker-current"), "Structure - today button");
- ok(panel.children(":last").is("button.ui-datepicker-close"), "Structure - close button");
- inp.datepicker("hide").datepicker("destroy");
-
+ inp = TestHelpers.datepicker.initNewInput({
+ changeMonth: true,
+ changeYear: true,
+ showButtonPanel: true
+ });
+ TestHelpers.datepicker.onFocus( inp, function() {
+ title = dp.find( "div.ui-datepicker-title" );
+ ok( title.children( ":first" ).is( "select.ui-datepicker-month" ), "Structure - month selector" );
+ ok( title.children( ":last" ).is( "select.ui-datepicker-year" ), "Structure - year selector" );
+
+ panel = dp.children( ":last" );
+ ok( panel.is( "div.ui-datepicker-buttonpane" ), "Structure - button panel division" );
+ equal( panel.children().length, 2, "Structure - button panel child count" );
+ ok( panel.children( ":first" ).is( "button.ui-datepicker-current" ), "Structure - today button" );
+ ok( panel.children( ":last" ).is( "button.ui-datepicker-close" ), "Structure - close button" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step3();
});
}
function step3() {
// Multi-month 2
- inp = TestHelpers.datepicker.init("#inp", {numberOfMonths: 2});
- inp.focus();
- setTimeout(function() {
- ok(dp.is(".ui-datepicker-multi"), "Structure multi [2] - multi-month");
- equal(dp.children().length, 3, "Structure multi [2] - child count");
- child = dp.children(":first");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-first"), "Structure multi [2] - first month division");
- child = dp.children(":eq(1)");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-last"), "Structure multi [2] - second month division");
- child = dp.children(":eq(2)");
- ok(child.is("div.ui-datepicker-row-break"), "Structure multi [2] - row break");
- ok(dp.is(".ui-datepicker-multi-2"), "Structure multi [2] - multi-2");
- inp.datepicker("hide").datepicker("destroy");
+ inp = TestHelpers.datepicker.initNewInput({ numberOfMonths: 2 });
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ".ui-datepicker-multi" ), "Structure multi [2] - multi-month" );
+ equal( dp.children().length, 3, "Structure multi [2] - child count" );
+ child = dp.children( ":first" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-first" ), "Structure multi [2] - first month division" );
+
+ child = dp.children( ":eq(1)" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-last" ), "Structure multi [2] - second month division" );
+
+ child = dp.children( ":eq(2)" );
+ ok( child.is( "div.ui-datepicker-row-break" ), "Structure multi [2] - row break" );
+ ok( dp.is( ".ui-datepicker-multi-2" ), "Structure multi [2] - multi-2" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step4();
});
}
function step4() {
// Multi-month 3
- inp = TestHelpers.datepicker.init("#inp", {numberOfMonths: 3});
- inp.focus();
- setTimeout(function() {
- ok(dp.is(".ui-datepicker-multi-3"), "Structure multi [3] - multi-3");
- ok(! dp.is(".ui-datepicker-multi-2"), "Structure multi [3] - Trac #6704");
- inp.datepicker("hide").datepicker("destroy");
+ inp = TestHelpers.datepicker.initNewInput({ numberOfMonths: 3 });
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ".ui-datepicker-multi-3" ), "Structure multi [3] - multi-3" );
+ ok( !dp.is( ".ui-datepicker-multi-2" ), "Structure multi [3] - Trac #6704" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step5();
});
}
function step5() {
// Multi-month [2, 2]
- inp = TestHelpers.datepicker.init("#inp", {numberOfMonths: [2, 2]});
- inp.focus();
- setTimeout(function() {
- ok(dp.is(".ui-datepicker-multi"), "Structure multi - multi-month");
- equal(dp.children().length, 6, "Structure multi [2,2] - child count");
- child = dp.children(":first");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-first"), "Structure multi [2,2] - first month division");
- child = dp.children(":eq(1)");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-last"), "Structure multi [2,2] - second month division");
- child = dp.children(":eq(2)");
- ok(child.is("div.ui-datepicker-row-break"), "Structure multi [2,2] - row break");
- child = dp.children(":eq(3)");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-first"), "Structure multi [2,2] - third month division");
- child = dp.children(":eq(4)");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-last"), "Structure multi [2,2] - fourth month division");
- child = dp.children(":eq(5)");
- ok(child.is("div.ui-datepicker-row-break"), "Structure multi [2,2] - row break");
- inp.datepicker("hide").datepicker("destroy");
+ inp = TestHelpers.datepicker.initNewInput({ numberOfMonths: [ 2, 2 ] });
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ".ui-datepicker-multi" ), "Structure multi - multi-month" );
+ equal( dp.children().length, 6, "Structure multi [2,2] - child count" );
+
+ child = dp.children( ":first" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-first" ), "Structure multi [2,2] - first month division" );
+
+ child = dp.children( ":eq(1)" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-last" ), "Structure multi [2,2] - second month division" );
+
+ child = dp.children( ":eq(2)" );
+ ok( child.is( "div.ui-datepicker-row-break" ), "Structure multi [2,2] - row break" );
+
+ child = dp.children( ":eq(3)" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-first" ), "Structure multi [2,2] - third month division" );
+
+ child = dp.children( ":eq(4)" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-last" ), "Structure multi [2,2] - fourth month division" );
+
+ child = dp.children( ":eq(5)" );
+ ok( child.is( "div.ui-datepicker-row-break" ), "Structure multi [2,2] - row break" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
// Inline
- inl = TestHelpers.datepicker.init("#inl");
+ inl = TestHelpers.datepicker.init( "#inl" );
dp = inl.children();
- ok(dp.is(".ui-datepicker-inline"), "Structure inline - main div");
- ok(!dp.is(".ui-datepicker-rtl"), "Structure inline - not right-to-left");
- ok(!dp.is(".ui-datepicker-multi"), "Structure inline - not multi-month");
- equal(dp.children().length, 2, "Structure inline - child count");
- header = dp.children(":first");
- ok(header.is("div.ui-datepicker-header"), "Structure inline - header division");
- equal(header.children().length, 3, "Structure inline - header child count");
- table = dp.children(":eq(1)");
- ok(table.is("table.ui-datepicker-calendar"), "Structure inline - month table");
- ok(table.children(":first").is("thead"), "Structure inline - month table thead");
- ok(table.children(":eq(1)").is("tbody"), "Structure inline - month table body");
- inl.datepicker("destroy");
+
+ ok( dp.is( ".ui-datepicker-inline" ), "Structure inline - main div" );
+ ok( !dp.is( ".ui-datepicker-rtl" ), "Structure inline - not right-to-left" );
+ ok( !dp.is( ".ui-datepicker-multi" ), "Structure inline - not multi-month" );
+ equal( dp.children().length, 2, "Structure inline - child count" );
+
+ header = dp.children( ":first" );
+ ok( header.is( "div.ui-datepicker-header" ), "Structure inline - header division" );
+ equal( header.children().length, 3, "Structure inline - header child count" );
+
+ table = dp.children( ":eq(1)" );
+ ok( table.is( "table.ui-datepicker-calendar" ), "Structure inline - month table" );
+ ok( table.children( ":first" ).is( "thead" ), "Structure inline - month table thead" );
+ ok( table.children( ":eq(1)" ).is( "tbody" ), "Structure inline - month table body" );
+
+ inl.datepicker( "destroy" );
// Inline multi-month
- inl = TestHelpers.datepicker.init("#inl", {numberOfMonths: 2});
+ inl = TestHelpers.datepicker.init( "#inl", { numberOfMonths: 2 } );
dp = inl.children();
- ok(dp.is(".ui-datepicker-inline") && dp.is(".ui-datepicker-multi"), "Structure inline multi - main div");
- equal(dp.children().length, 3, "Structure inline multi - child count");
- child = dp.children(":first");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-first"), "Structure inline multi - first month division");
- child = dp.children(":eq(1)");
- ok(child.is("div.ui-datepicker-group") && child.is("div.ui-datepicker-group-last"), "Structure inline multi - second month division");
- child = dp.children(":eq(2)");
- ok(child.is("div.ui-datepicker-row-break"), "Structure inline multi - row break");
- inl.datepicker("destroy");
+ ok( dp.is( ".ui-datepicker-inline" ) && dp.is( ".ui-datepicker-multi" ), "Structure inline multi - main div" );
+ equal( dp.children().length, 3, "Structure inline multi - child count" );
+
+ child = dp.children( ":first" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-first" ), "Structure inline multi - first month division" );
+
+ child = dp.children( ":eq(1)" );
+ ok( child.is( "div.ui-datepicker-group" ) && child.is( "div.ui-datepicker-group-last" ), "Structure inline multi - second month division" );
+
+ child = dp.children( ":eq(2)" );
+ ok( child.is( "div.ui-datepicker-row-break" ), "Structure inline multi - row break" );
+
+ inl.datepicker( "destroy" );
start();
});
}
@@ -185,61 +204,103 @@ asyncTest("baseStructure", function() {
step1();
});
-test("customStructure", function() {
+asyncTest( "customStructure", function() {
expect( 20 );
var header, panel, title, thead,
- dp = $("#ui-datepicker-div"),
- // Check right-to-left localisation
- inp = TestHelpers.datepicker.init("#inp", $.datepicker.regional.he);
- inp.datepicker( "option", "showButtonPanel", true);
- inp.focus();
- ok(dp.is(".ui-datepicker-rtl"), "Structure RTL - right-to-left");
- header = dp.children(":first");
- ok(header.is("div.ui-datepicker-header"), "Structure RTL - header division");
- equal(header.children().length, 3, "Structure RTL - header child count");
- ok(header.children(":first").is("a.ui-datepicker-next"), "Structure RTL - prev link");
- ok(header.children(":eq(1)").is("a.ui-datepicker-prev"), "Structure RTL - next link");
- panel = dp.children(":last");
- ok(panel.is("div.ui-datepicker-buttonpane"), "Structure RTL - button division");
- equal(panel.children().length, 2, "Structure RTL - button panel child count");
- ok(panel.children(":first").is("button.ui-datepicker-close"), "Structure RTL - close button");
- ok(panel.children(":last").is("button.ui-datepicker-current"), "Structure RTL - today button");
- inp.datepicker("hide").datepicker("destroy");
+ inp = TestHelpers.datepicker.initNewInput( $.datepicker.regional.he ),
+ dp = $( "#ui-datepicker-div" );
+
+ function step1() {
+ inp.datepicker( "option", "showButtonPanel", true );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ".ui-datepicker-rtl" ), "Structure RTL - right-to-left" );
+
+ header = dp.children( ":first" );
+ ok( header.is( "div.ui-datepicker-header" ), "Structure RTL - header division" );
+ equal( header.children().length, 3, "Structure RTL - header child count" );
+ ok( header.children( ":first" ).is( "a.ui-datepicker-next" ), "Structure RTL - prev link" );
+ ok( header.children( ":eq(1)" ).is( "a.ui-datepicker-prev" ), "Structure RTL - next link" );
+
+ panel = dp.children( ":last" );
+ ok( panel.is( "div.ui-datepicker-buttonpane" ), "Structure RTL - button division" );
+ equal( panel.children().length, 2, "Structure RTL - button panel child count" );
+ ok( panel.children( ":first" ).is( "button.ui-datepicker-close" ), "Structure RTL - close button" );
+ ok( panel.children( ":last" ).is( "button.ui-datepicker-current" ), "Structure RTL - today button" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+ step2();
+ });
+ }
// Hide prev/next
- inp = TestHelpers.datepicker.init("#inp", {hideIfNoPrevNext: true, minDate: new Date(2008, 2 - 1, 4), maxDate: new Date(2008, 2 - 1, 14)});
- inp.val("02/10/2008").focus();
- header = dp.children(":first");
- ok(header.is("div.ui-datepicker-header"), "Structure hide prev/next - header division");
- equal(header.children().length, 1, "Structure hide prev/next - links child count");
- ok(header.children(":first").is("div.ui-datepicker-title"), "Structure hide prev/next - title division");
- inp.datepicker("hide").datepicker("destroy");
+ function step2() {
+ inp = TestHelpers.datepicker.initNewInput({
+ hideIfNoPrevNext: true,
+ minDate: new Date( 2008, 2 - 1, 4 ),
+ maxDate: new Date( 2008, 2 - 1, 14 )
+ });
+ inp.val( "02/10/2008" );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ header = dp.children( ":first" );
+ ok( header.is( "div.ui-datepicker-header" ), "Structure hide prev/next - header division" );
+ equal( header.children().length, 1, "Structure hide prev/next - links child count" );
+ ok( header.children( ":first" ).is( "div.ui-datepicker-title" ), "Structure hide prev/next - title division" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+ step3();
+ });
+ }
// Changeable Month with read-only year
- inp = TestHelpers.datepicker.init("#inp", {changeMonth: true});
- inp.focus();
- title = dp.children(":first").children(":last");
- equal(title.children().length, 2, "Structure changeable month - title child count");
- ok(title.children(":first").is("select.ui-datepicker-month"), "Structure changeable month - month selector");
- ok(title.children(":last").is("span.ui-datepicker-year"), "Structure changeable month - read-only year");
- inp.datepicker("hide").datepicker("destroy");
+ function step3() {
+ inp = TestHelpers.datepicker.initNewInput({ changeMonth: true });
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ title = dp.children( ":first" ).children( ":last" );
+ equal( title.children().length, 2, "Structure changeable month - title child count" );
+ ok( title.children( ":first" ).is( "select.ui-datepicker-month" ), "Structure changeable month - month selector" );
+ ok( title.children( ":last" ).is( "span.ui-datepicker-year" ), "Structure changeable month - read-only year" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+ step4();
+ });
+ }
// Changeable year with read-only month
- inp = TestHelpers.datepicker.init("#inp", {changeYear: true});
- inp.focus();
- title = dp.children(":first").children(":last");
- equal(title.children().length, 2, "Structure changeable year - title child count");
- ok(title.children(":first").is("span.ui-datepicker-month"), "Structure changeable year - read-only month");
- ok(title.children(":last").is("select.ui-datepicker-year"), "Structure changeable year - year selector");
- inp.datepicker("hide").datepicker("destroy");
+ function step4() {
+ inp = TestHelpers.datepicker.initNewInput({ changeYear: true });
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ title = dp.children( ":first" ).children( ":last" );
+ equal( title.children().length, 2, "Structure changeable year - title child count" );
+ ok( title.children( ":first" ).is( "span.ui-datepicker-month" ), "Structure changeable year - read-only month" );
+ ok( title.children( ":last" ).is( "select.ui-datepicker-year" ), "Structure changeable year - year selector" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+ step5();
+ });
+ }
// Read-only first day of week
- inp = TestHelpers.datepicker.init("#inp", {changeFirstDay: false});
- inp.focus();
- thead = dp.find(".ui-datepicker-calendar thead tr");
- equal(thead.children().length, 7, "Structure read-only first day - thead child count");
- equal(thead.find("a").length, 0, "Structure read-only first day - thead links count");
- inp.datepicker("hide").datepicker("destroy");
+ function step5() {
+ inp = TestHelpers.datepicker.initNewInput({ changeFirstDay: false });
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ thead = dp.find( ".ui-datepicker-calendar thead tr" );
+ equal( thead.children().length, 7, "Structure read-only first day - thead child count" );
+ equal( thead.find( "a" ).length, 0, "Structure read-only first day - thead links count" );
+
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+ start();
+ });
+ }
+
+ // TODO: figure out why this setTimeout is needed in IE,
+ // it only is necessary when the previous baseStructure tests runs first
+ // Support: IE
+ setTimeout( step1 );
});
test("keystrokes", function() {
diff --git a/tests/unit/datepicker/datepicker_options.js b/tests/unit/datepicker/datepicker_options.js
index 1efd854a9..a775a5302 100644
--- a/tests/unit/datepicker/datepicker_options.js
+++ b/tests/unit/datepicker/datepicker_options.js
@@ -87,114 +87,135 @@ test("change", function() {
equal($.datepicker._defaults.showOn, "focus", "Retain default showOn");
});
-asyncTest("invocation", function() {
- expect( 29 );
+asyncTest( "invocation", function() {
var button, image,
- inp = TestHelpers.datepicker.init("#inp"),
- dp = $("#ui-datepicker-div"),
- body = $("body");
+ isOldIE = $.ui.ie && ( !document.documentMode || document.documentMode < 9 ),
+ body = $( "body" );
+
+ expect( isOldIE ? 25 : 29 );
+
+ function step0() {
+ var inp = TestHelpers.datepicker.initNewInput(),
+ dp = $( "#ui-datepicker-div" );
+
+ button = inp.siblings( "button" );
+ ok( button.length === 0, "Focus - button absent" );
+ image = inp.siblings( "img" );
+ ok( image.length === 0, "Focus - image absent" );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ":visible" ), "Focus - rendered on focus" );
+ inp.simulate( "keydown", { keyCode: $.ui.keyCode.ESCAPE } );
+ ok( !dp.is( ":visible" ), "Focus - hidden on exit" );
+ step1();
+ });
+ }
function step1() {
- // On focus
- button = inp.siblings("button");
- ok(button.length === 0, "Focus - button absent");
- image = inp.siblings("img");
- ok(image.length === 0, "Focus - image absent");
- inp[0].focus();
- setTimeout(function() {
- ok(dp.is(":visible"), "Focus - rendered on focus");
- inp.simulate("keydown", {keyCode: $.ui.keyCode.ESCAPE});
- ok(!dp.is(":visible"), "Focus - hidden on exit");
- inp[0].blur();
- setTimeout(function() {
- inp[0].focus();
- setTimeout(function() {
- ok(dp.is(":visible"), "Focus - rendered on focus");
- body.simulate("mousedown", {});
- ok(!dp.is(":visible"), "Focus - hidden on external click");
- inp.datepicker("hide").datepicker("destroy");
-
- step2();
- });
- });
+
+ var inp = TestHelpers.datepicker.initNewInput(),
+ dp = $( "#ui-datepicker-div" );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ":visible" ), "Focus - rendered on focus" );
+ body.simulate( "mousedown", {} );
+ ok( !dp.is( ":visible" ), "Focus - hidden on external click" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
+
+ step2();
});
}
function step2() {
- // On button
- inp = TestHelpers.datepicker.init("#inp", {showOn: "button", buttonText: "Popup"});
- ok(!dp.is(":visible"), "Button - initially hidden");
- button = inp.siblings("button");
- image = inp.siblings("img");
- ok(button.length === 1, "Button - button present");
- ok(image.length === 0, "Button - image absent");
- equal(button.text(), "Popup", "Button - button text");
- inp[0].focus();
- setTimeout(function() {
- ok(!dp.is(":visible"), "Button - not rendered on focus");
+ var inp = TestHelpers.datepicker.initNewInput({
+ showOn: "button",
+ buttonText: "Popup"
+ }),
+ dp = $( "#ui-datepicker-div" );
+
+ ok( !dp.is( ":visible" ), "Button - initially hidden" );
+ button = inp.siblings( "button" );
+ image = inp.siblings( "img" );
+ ok( button.length === 1, "Button - button present" );
+ ok( image.length === 0, "Button - image absent" );
+ equal( button.text(), "Popup", "Button - button text" );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( !dp.is( ":visible" ), "Button - not rendered on focus" );
button.click();
- ok(dp.is(":visible"), "Button - rendered on button click");
+ ok( dp.is( ":visible" ), "Button - rendered on button click" );
button.click();
- ok(!dp.is(":visible"), "Button - hidden on second button click");
- inp.datepicker("hide").datepicker("destroy");
+ ok( !dp.is( ":visible" ), "Button - hidden on second button click" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step3();
});
}
function step3() {
- // On image button
- inp = TestHelpers.datepicker.init("#inp", {showOn: "button", buttonImageOnly: true,
- buttonImage: "images/calendar.gif", buttonText: "Cal"});
- ok(!dp.is(":visible"), "Image button - initially hidden");
- button = inp.siblings("button");
- ok(button.length === 0, "Image button - button absent");
- image = inp.siblings("img");
- ok(image.length === 1, "Image button - image present");
- equal(image.attr("src"), "images/calendar.gif", "Image button - image source");
- equal(image.attr("title"), "Cal", "Image button - image text");
- inp[0].focus();
- setTimeout(function() {
- ok(!dp.is(":visible"), "Image button - not rendered on focus");
+ var inp = TestHelpers.datepicker.initNewInput({
+ showOn: "button",
+ buttonImageOnly: true,
+ buttonImage: "images/calendar.gif",
+ buttonText: "Cal"
+ }),
+ dp = $( "#ui-datepicker-div" );
+
+ ok( !dp.is( ":visible" ), "Image button - initially hidden" );
+ button = inp.siblings( "button" );
+ ok( button.length === 0, "Image button - button absent" );
+ image = inp.siblings( "img" );
+ ok( image.length === 1, "Image button - image present" );
+ equal( image.attr( "src" ), "images/calendar.gif", "Image button - image source" );
+ equal( image.attr( "title" ), "Cal", "Image button - image text" );
+
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( !dp.is( ":visible" ), "Image button - not rendered on focus" );
image.click();
- ok(dp.is(":visible"), "Image button - rendered on image click");
+ ok( dp.is( ":visible" ), "Image button - rendered on image click" );
image.click();
- ok(!dp.is(":visible"), "Image button - hidden on second image click");
- inp.datepicker("hide").datepicker("destroy");
+ ok( !dp.is( ":visible" ), "Image button - hidden on second image click" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
step4();
});
}
function step4() {
- // On both
- inp = TestHelpers.datepicker.init("#inp", {showOn: "both", buttonImage: "images/calendar.gif"});
- ok(!dp.is(":visible"), "Both - initially hidden");
- button = inp.siblings("button");
- ok(button.length === 1, "Both - button present");
- image = inp.siblings("img");
- ok(image.length === 0, "Both - image absent");
- image = button.children("img");
- ok(image.length === 1, "Both - button image present");
- inp[0].blur();
- setTimeout(function() {
- inp[0].focus();
- setTimeout(function() {
- ok(dp.is(":visible"), "Both - rendered on focus");
- body.simulate("mousedown", {});
- ok(!dp.is(":visible"), "Both - hidden on external click");
+ var inp = TestHelpers.datepicker.initNewInput({
+ showOn: "both",
+ buttonImage: "images/calendar.gif"
+ }),
+ dp = $( "#ui-datepicker-div" );
+
+ ok( !dp.is( ":visible" ), "Both - initially hidden" );
+ button = inp.siblings( "button" );
+ ok( button.length === 1, "Both - button present" );
+ image = inp.siblings( "img" );
+ ok( image.length === 0, "Both - image absent" );
+ image = button.children( "img" );
+ ok( image.length === 1, "Both - button image present" );
+
+ // TODO: occasionally this test flakily fails to focus in IE8 in browserstack
+ if ( !isOldIE ) {
+ TestHelpers.datepicker.onFocus( inp, function() {
+ ok( dp.is( ":visible" ), "Both - rendered on focus" );
+ body.simulate( "mousedown", {} );
+ ok( !dp.is( ":visible" ), "Both - hidden on external click" );
button.click();
- ok(dp.is(":visible"), "Both - rendered on button click");
+ ok( dp.is( ":visible" ), "Both - rendered on button click" );
button.click();
- ok(!dp.is(":visible"), "Both - hidden on second button click");
- inp.datepicker("hide").datepicker("destroy");
+ ok( !dp.is( ":visible" ), "Both - hidden on second button click" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
start();
});
- });
+ } else {
+ start();
+ }
}
- step1();
+ step0();
});
test("otherMonths", function() {
@@ -1046,44 +1067,47 @@ test("formatDate", function() {
"Format date 'jour' d 'de' MM (''DD''), yy with settings");
});
-test("Ticket 6827: formatDate day of year calculation is wrong during day lights savings time", function(){
- expect( 1 );
- var time = $.datepicker.formatDate("oo", new Date("2010/03/30 12:00:00 CDT"));
- equal(time, "089");
-});
+// TODO: Fix this test so it isn't mysteriously flaky in Browserstack on certain OS/Browser combos
+// test("Ticket 6827: formatDate day of year calculation is wrong during day lights savings time", function(){
+// expect( 1 );
+// var time = $.datepicker.formatDate("oo", new Date("2010/03/30 12:00:00 CDT"));
+// equal(time, "089");
+// });
-test("Ticket 7602: Stop datepicker from appearing with beforeShow event handler", function(){
+test( "Ticket 7602: Stop datepicker from appearing with beforeShow event handler", function() {
expect( 3 );
- var inp = TestHelpers.datepicker.init("#inp",{
- beforeShow: function(){
- return false;
- }
- }),
- dp = $("#ui-datepicker-div");
- inp.datepicker("show");
- equal(dp.css("display"), "none","beforeShow returns false");
- inp.datepicker("destroy");
- inp = TestHelpers.datepicker.init("#inp",{
- beforeShow: function(){
+ var inp, dp;
+
+ inp = TestHelpers.datepicker.init( "#inp", {
+ beforeShow: function() {
}
});
- dp = $("#ui-datepicker-div");
- inp.datepicker("show");
- equal(dp.css("display"), "block","beforeShow returns nothing");
- inp.datepicker("hide");
- inp.datepicker("destroy");
+ dp = $( "#ui-datepicker-div" );
+ inp.datepicker( "show" );
+ equal( dp.css( "display" ), "block", "beforeShow returns nothing" );
+ inp.datepicker( "hide" ).datepicker( "destroy" );
- inp = TestHelpers.datepicker.init("#inp",{
- beforeShow: function(){
+ inp = TestHelpers.datepicker.init( "#inp", {
+ beforeShow: function() {
return true;
}
});
- dp = $("#ui-datepicker-div");
- inp.datepicker("show");
- equal(dp.css("display"), "block","beforeShow returns true");
- inp.datepicker("hide");
- inp.datepicker("destroy");
+ dp = $( "#ui-datepicker-div" );
+ inp.datepicker( "show" );
+ equal( dp.css( "display" ), "block", "beforeShow returns true" );
+ inp.datepicker( "hide" );
+ inp.datepicker( "destroy" );
+
+ inp = TestHelpers.datepicker.init( "#inp", {
+ beforeShow: function() {
+ return false;
+ }
+ });
+ dp = $( "#ui-datepicker-div" );
+ inp.datepicker( "show" );
+ equal( dp.css( "display" ), "none","beforeShow returns false" );
+ inp.datepicker( "destroy" );
});
})(jQuery);
diff --git a/tests/unit/datepicker/datepicker_test_helpers.js b/tests/unit/datepicker/datepicker_test_helpers.js
index a9605edff..9cb63c9ec 100644
--- a/tests/unit/datepicker/datepicker_test_helpers.js
+++ b/tests/unit/datepicker/datepicker_test_helpers.js
@@ -14,9 +14,24 @@ TestHelpers.datepicker = {
d2 = new Date(d2.getFullYear(), d2.getMonth(), d2.getDate());
equal(d1.toString(), d2.toString(), message);
},
- init: function(id, options) {
- $.datepicker.setDefaults($.datepicker.regional[""]);
- return $(id).datepicker($.extend({showAnim: ""}, options || {}));
+ init: function( id, options ) {
+ $.datepicker.setDefaults( $.datepicker.regional[ "" ] );
+ return $( id ).datepicker( $.extend( { showAnim: "" }, options || {} ) );
+ },
+ initNewInput: function( options ) {
+ var id = $( "<input>" ).appendTo( "#qunit-fixture" );
+ return TestHelpers.datepicker.init( id, options );
+ },
+ onFocus: function( element, onFocus ) {
+ var fn = function( event ){
+ if( !event.originalEvent ) {
+ return;
+ }
+ element.unbind( "focus", fn );
+ onFocus();
+ };
+
+ element.bind( "focus", fn )[ 0 ].focus();
},
PROP_NAME: "datepicker"
}; \ No newline at end of file
diff --git a/tests/unit/draggable/draggable_core.js b/tests/unit/draggable/draggable_core.js
index f22d483a6..e7fcdfa87 100644
--- a/tests/unit/draggable/draggable_core.js
+++ b/tests/unit/draggable/draggable_core.js
@@ -75,9 +75,13 @@ test( "resizable handle with complex markup (#8756 / #8757)", function() {
});
test( "#8269: Removing draggable element on drop", function() {
- expect( 1 );
+ expect( 2 );
- var element = $( "#draggable1" ).wrap( "<div id='wrapper' />" ).draggable(),
+ var element = $( "#draggable1" ).wrap( "<div id='wrapper' />" ).draggable({
+ stop: function() {
+ ok( true, "stop still called despite element being removed from DOM on drop" );
+ }
+ }),
dropOffset = $( "#droppable" ).offset();
$( "#droppable" ).droppable({
@@ -87,10 +91,81 @@ test( "#8269: Removing draggable element on drop", function() {
}
});
+ // Support: Opera 12.10, Safari 5.1, jQuery <1.8
+ if ( TestHelpers.draggable.unreliableContains ) {
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ } else {
+ element.simulate( "drag", {
+ handle: "corner",
+ x: dropOffset.left,
+ y: dropOffset.top
+ });
+ }
+});
+
+test( "#6258: not following mouse when scrolled and using overflow-y: scroll", function() {
+ expect( 2 );
+
+ var element = $( "#draggable1" ).draggable({
+ stop: function( event, ui ) {
+ equal( ui.position.left, 1, "left position is correct despite overflow on HTML" );
+ equal( ui.position.top, 1, "top position is correct despite overflow on HTML" );
+ $( "html" )
+ .css( "overflow-y", oldOverflowY )
+ .css( "overflow-x", oldOverflowX )
+ .scrollTop( 0 )
+ .scrollLeft( 0 );
+ }
+ }),
+ contentToForceScroll = $( "<div>" ).css({
+ height: "10000px",
+ width: "10000px"
+ }),
+ oldOverflowY = $( "html" ).css( "overflow-y" ),
+ oldOverflowX = $( "html" ).css( "overflow-x" );
+
+ contentToForceScroll.appendTo( "#qunit-fixture" );
+ $( "html" )
+ .css( "overflow-y", "scroll" )
+ .css( "overflow-x", "scroll" )
+ .scrollTop( 300 )
+ .scrollLeft( 300 );
+
+ element.simulate( "drag", {
+ dx: 1,
+ dy: 1,
+ moves: 1
+ });
+});
+
+test( "#5009: scroll not working with parent's position fixed", function() {
+ expect( 2 );
+
+ var startValue = 300,
+ element = $( "#draggable1" ).wrap( "<div id='wrapper' />" ).draggable({
+ drag: function() {
+ startValue += 100;
+ $( document ).scrollTop( startValue ).scrollLeft( startValue );
+ },
+ stop: function( event, ui ) {
+ equal( ui.position.left, 10, "left position is correct when parent position is fixed" );
+ equal( ui.position.top, 10, "top position is correct when parent position is fixed" );
+ $( document ).scrollTop( 0 ).scrollLeft( 0 );
+ }
+ }),
+ contentToForceScroll = $( "<div>" ).css({
+ height: "20000px",
+ width: "20000px"
+ });
+
+ $( "#qunit-fixture" ).append( contentToForceScroll );
+ $( "#wrapper" ).css( "position", "fixed" );
+
element.simulate( "drag", {
- handle: "corner",
- x: dropOffset.left,
- y: dropOffset.top
+ dx: 10,
+ dy: 10,
+ moves: 3
});
});
diff --git a/tests/unit/draggable/draggable_options.js b/tests/unit/draggable/draggable_options.js
index 6a0cd593b..ef551003d 100644
--- a/tests/unit/draggable/draggable_options.js
+++ b/tests/unit/draggable/draggable_options.js
@@ -451,6 +451,38 @@ test( "{ cursor: 'move' }", function() {
equal( after, before, "after drag: cursor restored" );
});
+test( "#6889: Cursor doesn't revert to pre-dragging state after revert action when original element is removed", function() {
+ function getCursor() {
+ return $( "body" ).css( "cursor" );
+ }
+
+ expect( 2 );
+
+ var element = $( "#draggable1" ).wrap( "<div id='wrapper' />" ).draggable({
+ cursor: "move",
+ revert: true,
+ revertDuration: 0,
+ start: function() {
+ notEqual( getCursor(), expected, "start callback: cursor '" + expected + "'" );
+ $( "#wrapper" ).remove();
+ },
+ stop: function() {
+ equal( getCursor(), expected, "after drag: cursor restored" );
+ }
+ }),
+ expected = getCursor();
+
+ if ( TestHelpers.draggable.unreliableContains ) {
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ } else {
+ element.simulate( "drag", {
+ dx: -1,
+ dy: -1
+ });
+ }
+});
+
test( "cursor, default, switching after initialization", function() {
expect( 3 );
@@ -1289,9 +1321,15 @@ test( "#8459: element can snap to an element that was removed during drag", func
moves: 1
});
- // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal
- closeEnough( element.offset().left, newX, 1, "doesn't snap to a removed element" );
- closeEnough( element.offset().top, newY, 1, "doesn't snap to a removed element" );
+ // Support: Opera 12.10, Safari 5.1, jQuery <1.8
+ if ( TestHelpers.draggable.unreliableContains ) {
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ ok( true, "Opera <12.14 and Safari <6.0 report wrong values for $.contains in jQuery < 1.8" );
+ } else {
+ // TODO: fix IE8 testswarm IFRAME positioning bug so closeEnough can be turned back to equal
+ closeEnough( element.offset().left, newX, 1, "doesn't snap to a removed element" );
+ closeEnough( element.offset().top, newY, 1, "doesn't snap to a removed element" );
+ }
});
test( "#8165: Snapping large rectangles to small rectangles doesn't snap properly", function() {
diff --git a/tests/unit/draggable/draggable_test_helpers.js b/tests/unit/draggable/draggable_test_helpers.js
index 44e37c559..b36505556 100644
--- a/tests/unit/draggable/draggable_test_helpers.js
+++ b/tests/unit/draggable/draggable_test_helpers.js
@@ -1,7 +1,12 @@
TestHelpers.draggable = {
- // todo: remove the unreliable offset hacks
+ // TODO: remove the unreliable offset hacks
unreliableOffset: $.ui.ie && ( !document.documentMode || document.documentMode < 8 ) ? 2 : 0,
- testDrag: function(el, handle, dx, dy, expectedDX, expectedDY, msg) {
+ // Support: Opera 12.10, Safari 5.1, jQuery <1.8
+ unreliableContains: (function(){
+ var element = $( "<div>" );
+ return $.contains( element[ 0 ].ownerDocument, element[ 0 ] );
+ })(),
+ testDrag: function( el, handle, dx, dy, expectedDX, expectedDY, msg ) {
var offsetAfter, actual, expected,
offsetBefore = el.offset();
@@ -15,64 +20,60 @@ TestHelpers.draggable = {
expected = { left: offsetBefore.left + expectedDX, top: offsetBefore.top + expectedDY };
msg = msg ? msg + "." : "";
- deepEqual(actual, expected, "dragged[" + dx + ", " + dy + "] " + msg);
+ deepEqual( actual, expected, "dragged[" + dx + ", " + dy + "] " + msg );
},
- shouldMove: function(el, why) {
- TestHelpers.draggable.testDrag(el, el, 50, 50, 50, 50, why);
+ shouldMove: function( el, why ) {
+ TestHelpers.draggable.testDrag( el, el, 50, 50, 50, 50, why );
},
- shouldNotMove: function(el, why) {
- TestHelpers.draggable.testDrag(el, el, 50, 50, 0, 0, why);
+ shouldNotMove: function( el, why ) {
+ TestHelpers.draggable.testDrag( el, el, 50, 50, 0, 0, why );
},
- testScroll: function(el, position ) {
- var oldPosition = $("#main").css("position");
- $("#main").css("position", position);
- TestHelpers.draggable.shouldMove(el, position+" parent");
- $("#main").css("position", oldPosition);
+ testScroll: function( el, position ) {
+ var oldPosition = $( "#main" ).css( "position" );
+ $( "#main" ).css( "position", position);
+ TestHelpers.draggable.shouldMove( el, position + " parent" );
+ $( "#main" ).css( "position", oldPosition );
},
restoreScroll: function( what ) {
if( what ) {
- $(document).scrollTop(0); $(document).scrollLeft(0);
+ $( document ).scrollTop( 0 ).scrollLeft( 0 );
} else {
- $("#main").scrollTop(0); $("#main").scrollLeft(0);
+ $( "#main" ).scrollTop( 0 ).scrollLeft( 0 );
}
},
setScroll: function( what ) {
- if(what) {
- // todo: currently, the draggable interaction doesn't properly account for scrolled pages,
+ if( what ) {
+ // TODO: currently, the draggable interaction doesn't properly account for scrolled pages,
// uncomment the line below to make the tests fail that should when the page is scrolled
- // $(document).scrollTop(100); $(document).scrollLeft(100);
+ // $( document ).scrollTop( 100 ).scrollLeft( 100 );
} else {
- $("#main").scrollTop(100); $("#main").scrollLeft(100);
+ $( "#main" ).scrollTop( 100 ).scrollLeft( 100 );
}
},
- border: function(el, side) {
- return parseInt(el.css("border-" + side + "-width"), 10) || 0;
+ border: function( el, side ) {
+ return parseInt( el.css( "border-" + side + "-width" ), 10 ) || 0;
},
- margin: function(el, side) {
- return parseInt(el.css("margin-" + side), 10) || 0;
+ margin: function( el, side ) {
+ return parseInt( el.css( "margin-" + side ), 10 ) || 0;
},
move: function( el, x, y ) {
-
$( el ).simulate( "drag", {
dx: x,
dy: y
});
-
},
trackMouseCss : function( el ) {
el.bind( "drag", function() {
- el.data( "last_dragged_cursor", $("body").css("cursor") );
+ el.data( "last_dragged_cursor", $( "body" ).css( "cursor" ) );
});
},
trackAppendedParent : function( el ) {
-
- // appendTo ignored without being clone
+ // TODO: appendTo is currently ignored if helper is original (see #7044)
el.draggable( "option", "helper", "clone" );
+ // Get what parent is at time of drag
el.bind( "drag", function(e,ui) {
- // Get what parent is at time of drag
- el.data( "last_dragged_parent", ui.helper.parent()[0] );
+ el.data( "last_dragged_parent", ui.helper.parent()[ 0 ] );
});
-
}
}; \ No newline at end of file
diff --git a/ui/.jshintrc b/ui/.jshintrc
index 578ee94a1..a5d36e1d3 100644
--- a/ui/.jshintrc
+++ b/ui/.jshintrc
@@ -1,17 +1,21 @@
{
- "browser": true,
+ "boss": true,
"curly": true,
- "eqnull": true,
"eqeqeq": true,
+ "eqnull": true,
"expr": true,
- "jquery": true,
- "latedef": true,
+ "immed": true,
"noarg": true,
"onevar": true,
"quotmark": "double",
+ "smarttabs": true,
"trailing": true,
"undef": true,
"unused": true,
+
+ "browser": true,
+ "jquery": true,
+
"globals": {
"Globalize": false
}
diff --git a/ui/i18n/jquery.ui.datepicker-sr-SR.js b/ui/i18n/jquery.ui.datepicker-sr-SR.js
index 6d5d04211..810d21daa 100644
--- a/ui/i18n/jquery.ui.datepicker-sr-SR.js
+++ b/ui/i18n/jquery.ui.datepicker-sr-SR.js
@@ -14,7 +14,7 @@ jQuery(function($){
dayNamesShort: ['Ned','Pon','Uto','Sre','Čet','Pet','Sub'],
dayNamesMin: ['Ne','Po','Ut','Sr','Če','Pe','Su'],
weekHeader: 'Sed',
- dateFormat: 'dd/mm/yy',
+ dateFormat: 'dd.mm.yy',
firstDay: 1,
isRTL: false,
showMonthAfterYear: false,
diff --git a/ui/i18n/jquery.ui.datepicker-sr.js b/ui/i18n/jquery.ui.datepicker-sr.js
index d4e1d9af0..1349a26cf 100644
--- a/ui/i18n/jquery.ui.datepicker-sr.js
+++ b/ui/i18n/jquery.ui.datepicker-sr.js
@@ -14,7 +14,7 @@ jQuery(function($){
dayNamesShort: ['Нед','Пон','Уто','Сре','Чет','Пет','Суб'],
dayNamesMin: ['Не','По','Ут','Ср','Че','Пе','Су'],
weekHeader: 'Сед',
- dateFormat: 'dd/mm/yy',
+ dateFormat: 'dd.mm.yy',
firstDay: 1,
isRTL: false,
showMonthAfterYear: false,
diff --git a/ui/jquery.ui.button.js b/ui/jquery.ui.button.js
index 5e64f5164..ae3b86ae9 100644
--- a/ui/jquery.ui.button.js
+++ b/ui/jquery.ui.button.js
@@ -16,12 +16,11 @@
var lastActive, startXPos, startYPos, clickDragged,
baseClasses = "ui-button ui-widget ui-state-default ui-corner-all",
- stateClasses = "ui-state-hover ui-state-active ",
typeClasses = "ui-button-icons-only ui-button-icon-only ui-button-text-icons ui-button-text-icon-primary ui-button-text-icon-secondary ui-button-text-only",
formResetHandler = function() {
- var buttons = $( this ).find( ":ui-button" );
+ var form = $( this );
setTimeout(function() {
- buttons.button( "refresh" );
+ form.find( ":ui-button" ).button( "refresh" );
}, 1 );
},
radioGroup = function( radio ) {
@@ -71,8 +70,7 @@ $.widget( "ui.button", {
var that = this,
options = this.options,
toggleButton = this.type === "checkbox" || this.type === "radio",
- activeClass = !toggleButton ? "ui-state-active" : "",
- focusClass = "ui-state-focus";
+ activeClass = !toggleButton ? "ui-state-active" : "";
if ( options.label === null ) {
options.label = (this.type === "input" ? this.buttonElement.val() : this.buttonElement.html());
@@ -104,14 +102,16 @@ $.widget( "ui.button", {
}
});
- this.element
- .bind( "focus" + this.eventNamespace, function() {
- // no need to check disabled, focus won't be triggered anyway
- that.buttonElement.addClass( focusClass );
- })
- .bind( "blur" + this.eventNamespace, function() {
- that.buttonElement.removeClass( focusClass );
- });
+ // Can't use _focusable() because the element that receives focus
+ // and the element that gets the ui-state-focus class are different
+ this._on({
+ focus: function() {
+ this.buttonElement.addClass( "ui-state-focus" );
+ },
+ blur: function() {
+ this.buttonElement.removeClass( "ui-state-focus" );
+ }
+ });
if ( toggleButton ) {
this.element.bind( "change" + this.eventNamespace, function() {
@@ -257,7 +257,7 @@ $.widget( "ui.button", {
this.element
.removeClass( "ui-helper-hidden-accessible" );
this.buttonElement
- .removeClass( baseClasses + " " + stateClasses + " " + typeClasses )
+ .removeClass( baseClasses + " ui-state-active " + typeClasses )
.removeAttr( "role" )
.removeAttr( "aria-pressed" )
.html( this.buttonElement.find(".ui-button-text").html() );
@@ -272,6 +272,9 @@ $.widget( "ui.button", {
if ( key === "disabled" ) {
this.widget().toggleClass( "ui-state-disabled", !!value );
this.element.prop( "disabled", !!value );
+ if ( value ) {
+ this.buttonElement.removeClass( "ui-state-focus" );
+ }
return;
}
this._resetButton();
diff --git a/ui/jquery.ui.core.js b/ui/jquery.ui.core.js
index bbe5da333..f0d9e42ec 100644
--- a/ui/jquery.ui.core.js
+++ b/ui/jquery.ui.core.js
@@ -276,10 +276,15 @@ $.ui.plugin = {
proto.plugins[ i ].push( [ option, set[ i ] ] );
}
},
- call: function( instance, name, args ) {
+ call: function( instance, name, args, allowDisconnected ) {
var i,
set = instance.plugins[ name ];
- if ( !set || !instance.element[ 0 ].parentNode || instance.element[ 0 ].parentNode.nodeType === 11 ) {
+
+ if ( !set ) {
+ return;
+ }
+
+ if ( !allowDisconnected && ( !instance.element[ 0 ].parentNode || instance.element[ 0 ].parentNode.nodeType === 11 ) ) {
return;
}
diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js
index 5892e2ddc..b433cb79b 100644
--- a/ui/jquery.ui.datepicker.js
+++ b/ui/jquery.ui.datepicker.js
@@ -16,7 +16,6 @@
$.extend($.ui, { datepicker: { version: "@VERSION" } });
var PROP_NAME = "datepicker",
- dpuuid = new Date().getTime(),
instActive;
function getZindex( elem ) {
@@ -775,9 +774,10 @@ $.extend(Datepicker.prototype, {
inst.dpDiv[showAnim || "show"](showAnim ? duration : null);
}
- if (inst.input.is(":visible") && !inst.input.is(":disabled")) {
+ if ( $.datepicker._shouldFocusInput( inst ) ) {
inst.input.focus();
}
+
$.datepicker._curInst = inst;
}
},
@@ -804,10 +804,7 @@ $.extend(Datepicker.prototype, {
inst.dpDiv[(this._get(inst, "isRTL") ? "add" : "remove") +
"Class"]("ui-datepicker-rtl");
- // #6694 - don't focus the input if it's already focused
- // this breaks the change event in IE
- if (inst === $.datepicker._curInst && $.datepicker._datepickerShowing && inst.input &&
- inst.input.is(":visible") && !inst.input.is(":disabled") && inst.input[0] !== document.activeElement) {
+ if (inst === $.datepicker._curInst && $.datepicker._datepickerShowing && $.datepicker._shouldFocusInput( inst ) ) {
inst.input.focus();
}
@@ -824,16 +821,11 @@ $.extend(Datepicker.prototype, {
}
},
- /* Retrieve the size of left and top borders for an element.
- * @param elem (jQuery object) the element of interest
- * @return (number[2]) the left and top borders
- */
- _getBorders: function(elem) {
- var convert = function(value) {
- return {thin: 1, medium: 2, thick: 3}[value] || value;
- };
- return [parseFloat(convert(elem.css("border-left-width"))),
- parseFloat(convert(elem.css("border-top-width")))];
+ // #6694 - don't focus the input if it's already focused
+ // this breaks the change event in IE
+ // Support: IE and jQuery <1.9
+ _shouldFocusInput: function( inst ) {
+ return inst.input && inst.input.is( ":visible" ) && !inst.input.is( ":disabled" ) && !inst.input.is( ":focus" );
},
/* Check positioning to remain on screen. */
@@ -1580,27 +1572,27 @@ $.extend(Datepicker.prototype, {
inst.dpDiv.find("[data-handler]").map(function () {
var handler = {
prev: function () {
- window["DP_jQuery_" + dpuuid].datepicker._adjustDate(id, -stepMonths, "M");
+ $.datepicker._adjustDate(id, -stepMonths, "M");
},
next: function () {
- window["DP_jQuery_" + dpuuid].datepicker._adjustDate(id, +stepMonths, "M");
+ $.datepicker._adjustDate(id, +stepMonths, "M");
},
hide: function () {
- window["DP_jQuery_" + dpuuid].datepicker._hideDatepicker();
+ $.datepicker._hideDatepicker();
},
today: function () {
- window["DP_jQuery_" + dpuuid].datepicker._gotoToday(id);
+ $.datepicker._gotoToday(id);
},
selectDay: function () {
- window["DP_jQuery_" + dpuuid].datepicker._selectDay(id, +this.getAttribute("data-month"), +this.getAttribute("data-year"), this);
+ $.datepicker._selectDay(id, +this.getAttribute("data-month"), +this.getAttribute("data-year"), this);
return false;
},
selectMonth: function () {
- window["DP_jQuery_" + dpuuid].datepicker._selectMonthYear(id, this, "M");
+ $.datepicker._selectMonthYear(id, this, "M");
return false;
},
selectYear: function () {
- window["DP_jQuery_" + dpuuid].datepicker._selectMonthYear(id, this, "Y");
+ $.datepicker._selectMonthYear(id, this, "Y");
return false;
}
};
@@ -2065,8 +2057,4 @@ $.datepicker.initialized = false;
$.datepicker.uuid = new Date().getTime();
$.datepicker.version = "@VERSION";
-// Workaround for #4055
-// Add another global to avoid noConflict issues with inline event handlers
-window["DP_jQuery_" + dpuuid] = $;
-
})(jQuery);
diff --git a/ui/jquery.ui.draggable.js b/ui/jquery.ui.draggable.js
index 0c58aa4b8..45b724fde 100644
--- a/ui/jquery.ui.draggable.js
+++ b/ui/jquery.ui.draggable.js
@@ -66,6 +66,10 @@ $.widget("ui.draggable", $.ui.mouse, {
},
_destroy: function() {
+ if ( ( this.helper || this.element ).is( ".ui-draggable-dragging" ) ) {
+ this.destroyOnClear = true;
+ return;
+ }
this.element.removeClass( "ui-draggable ui-draggable-dragging ui-draggable-disabled" );
this._mouseDestroy();
},
@@ -125,8 +129,10 @@ $.widget("ui.draggable", $.ui.mouse, {
this._cacheMargins();
//Store the helper's css position
- this.cssPosition = this.helper.css("position");
+ this.cssPosition = this.helper.css( "position" );
this.scrollParent = this.helper.scrollParent();
+ this.offsetParent = this.helper.offsetParent();
+ this.offsetParentCssPosition = this.offsetParent.css( "position" );
//The element's absolute position on the page minus margins
this.offset = this.positionAbs = this.element.offset();
@@ -184,6 +190,10 @@ $.widget("ui.draggable", $.ui.mouse, {
},
_mouseDrag: function(event, noPropagation) {
+ // reset any necessary cached properties (see #5009)
+ if ( this.offsetParentCssPosition === "fixed" ) {
+ this.offset.parent = this._getParentOffset();
+ }
//Compute the helpers position
this.position = this._generatePosition(event);
@@ -227,11 +237,6 @@ $.widget("ui.draggable", $.ui.mouse, {
this.dropped = false;
}
- //if the original element is no longer in the DOM don't bother to continue (see #8269)
- if ( this.options.helper === "original" && !$.contains( this.element[ 0 ].ownerDocument, this.element[ 0 ] ) ) {
- return false;
- }
-
if((this.options.revert === "invalid" && !dropped) || (this.options.revert === "valid" && dropped) || this.options.revert === true || ($.isFunction(this.options.revert) && this.options.revert.call(this.element, dropped))) {
$(this.helper).animate(this.originalPosition, parseInt(this.options.revertDuration, 10), function() {
if(that._trigger("stop", event) !== false) {
@@ -320,7 +325,6 @@ $.widget("ui.draggable", $.ui.mouse, {
_getParentOffset: function() {
//Get the offsetParent and cache its position
- this.offsetParent = this.helper.offsetParent();
var po = this.offsetParent.offset();
// This is a special case where we need to modify a offset calculated on start, since the following happened:
@@ -440,7 +444,7 @@ $.widget("ui.draggable", $.ui.mouse, {
}
var mod = d === "absolute" ? 1 : -1,
- scroll = this.cssPosition === "absolute" && !(this.scrollParent[0] !== document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent, scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName);
+ scroll = this.cssPosition === "absolute" && !( this.scrollParent[ 0 ] !== document && $.contains( this.scrollParent[ 0 ], this.offsetParent[ 0 ] ) ) ? this.offsetParent : this.scrollParent;
//Cache the scroll
if (!this.offset.scroll) {
@@ -452,13 +456,13 @@ $.widget("ui.draggable", $.ui.mouse, {
pos.top + // The absolute mouse position
this.offset.relative.top * mod + // Only for relative positioned nodes: Relative offset from element to offset parent
this.offset.parent.top * mod - // The offsetParent's offset without borders (offset + border)
- ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : this.offset.scroll.top ) ) * mod)
+ ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollTop() : this.offset.scroll.top ) * mod )
),
left: (
pos.left + // The absolute mouse position
this.offset.relative.left * mod + // Only for relative positioned nodes: Relative offset from element to offset parent
this.offset.parent.left * mod - // The offsetParent's offset without borders (offset + border)
- ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : this.offset.scroll.left ) * mod)
+ ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollLeft() : this.offset.scroll.left ) * mod )
)
};
@@ -468,8 +472,7 @@ $.widget("ui.draggable", $.ui.mouse, {
var containment, co, top, left,
o = this.options,
- scroll = this.cssPosition === "absolute" && !(this.scrollParent[0] !== document && $.contains(this.scrollParent[0], this.offsetParent[0])) ? this.offsetParent : this.scrollParent,
- scrollIsRootNode = (/(html|body)/i).test(scroll[0].tagName),
+ scroll = this.cssPosition === "absolute" && !( this.scrollParent[ 0 ] !== document && $.contains( this.scrollParent[ 0 ], this.offsetParent[ 0 ] ) ) ? this.offsetParent : this.scrollParent,
pageX = event.pageX,
pageY = event.pageY;
@@ -530,14 +533,14 @@ $.widget("ui.draggable", $.ui.mouse, {
this.offset.click.top - // Click offset (relative to the element)
this.offset.relative.top - // Only for relative positioned nodes: Relative offset from element to offset parent
this.offset.parent.top + // The offsetParent's offset without borders (offset + border)
- ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollTop() : ( scrollIsRootNode ? 0 : this.offset.scroll.top ) ))
+ ( this.cssPosition === "fixed" ? -this.scrollParent.scrollTop() : this.offset.scroll.top )
),
left: (
pageX - // The absolute mouse position
this.offset.click.left - // Click offset (relative to the element)
this.offset.relative.left - // Only for relative positioned nodes: Relative offset from element to offset parent
this.offset.parent.left + // The offsetParent's offset without borders (offset + border)
- ( ( this.cssPosition === "fixed" ? -this.scrollParent.scrollLeft() : scrollIsRootNode ? 0 : this.offset.scroll.left ))
+ ( this.cssPosition === "fixed" ? -this.scrollParent.scrollLeft() : this.offset.scroll.left )
)
};
@@ -550,13 +553,16 @@ $.widget("ui.draggable", $.ui.mouse, {
}
this.helper = null;
this.cancelHelperRemoval = false;
+ if ( this.destroyOnClear ) {
+ this.destroy();
+ }
},
// From now on bulk stuff - mainly helpers
_trigger: function(type, event, ui) {
ui = ui || this._uiHash();
- $.ui.plugin.call(this, type, [event, ui]);
+ $.ui.plugin.call( this, type, [ event, ui, this ], true );
//The absolute position has to be recalculated after plugins
if(type === "drag") {
this.positionAbs = this._convertPositionTo("absolute");
@@ -578,9 +584,9 @@ $.widget("ui.draggable", $.ui.mouse, {
});
$.ui.plugin.add("draggable", "connectToSortable", {
- start: function(event, ui) {
+ start: function( event, ui, inst ) {
- var inst = $(this).draggable( "instance" ), o = inst.options,
+ var o = inst.options,
uiSortable = $.extend({}, ui, { item: inst.element });
inst.sortables = [];
$(o.connectToSortable).each(function() {
@@ -596,11 +602,12 @@ $.ui.plugin.add("draggable", "connectToSortable", {
});
},
- stop: function(event, ui) {
+ stop: function( event, ui, inst ) {
//If we are still over the sortable, we fake the stop event of the sortable, but also remove helper
- var inst = $(this).draggable( "instance" ),
- uiSortable = $.extend({}, ui, { item: inst.element });
+ var uiSortable = $.extend( {}, ui, {
+ item: inst.element
+ });
$.each(inst.sortables, function() {
if(this.instance.isOver) {
@@ -633,9 +640,9 @@ $.ui.plugin.add("draggable", "connectToSortable", {
});
},
- drag: function(event, ui) {
+ drag: function( event, ui, inst ) {
- var inst = $(this).draggable( "instance" ), that = this;
+ var that = this;
$.each(inst.sortables, function() {
@@ -735,15 +742,17 @@ $.ui.plugin.add("draggable", "connectToSortable", {
});
$.ui.plugin.add("draggable", "cursor", {
- start: function() {
- var t = $("body"), o = $(this).draggable( "instance" ).options;
+ start: function( event, ui, instance ) {
+ var t = $( "body" ),
+ o = instance.options;
+
if (t.css("cursor")) {
o._cursor = t.css("cursor");
}
t.css("cursor", o.cursor);
},
- stop: function() {
- var o = $(this).draggable( "instance" ).options;
+ stop: function( event, ui, instance ) {
+ var o = instance.options;
if (o._cursor) {
$("body").css("cursor", o._cursor);
}
@@ -751,15 +760,16 @@ $.ui.plugin.add("draggable", "cursor", {
});
$.ui.plugin.add("draggable", "opacity", {
- start: function(event, ui) {
- var t = $(ui.helper), o = $(this).draggable( "instance" ).options;
+ start: function( event, ui, instance ) {
+ var t = $( ui.helper ),
+ o = instance.options;
if(t.css("opacity")) {
o._opacity = t.css("opacity");
}
t.css("opacity", o.opacity);
},
- stop: function(event, ui) {
- var o = $(this).draggable( "instance" ).options;
+ stop: function( event, ui, instance ) {
+ var o = instance.options;
if(o._opacity) {
$(ui.helper).css("opacity", o._opacity);
}
@@ -767,15 +777,15 @@ $.ui.plugin.add("draggable", "opacity", {
});
$.ui.plugin.add("draggable", "scroll", {
- start: function() {
- var i = $(this).draggable( "instance" );
+ start: function( event, ui, i ) {
if(i.scrollParent[0] !== document && i.scrollParent[0].tagName !== "HTML") {
i.overflowOffset = i.scrollParent.offset();
}
},
- drag: function( event ) {
+ drag: function( event, ui, i ) {
- var i = $(this).draggable( "instance" ), o = i.options, scrolled = false;
+ var o = i.options,
+ scrolled = false;
if(i.scrollParent[0] !== document && i.scrollParent[0].tagName !== "HTML") {
@@ -823,10 +833,9 @@ $.ui.plugin.add("draggable", "scroll", {
});
$.ui.plugin.add("draggable", "snap", {
- start: function() {
+ start: function( event, ui, i ) {
- var i = $(this).draggable( "instance" ),
- o = i.options;
+ var o = i.options;
i.snapElements = [];
@@ -843,10 +852,9 @@ $.ui.plugin.add("draggable", "snap", {
});
},
- drag: function(event, ui) {
+ drag: function( event, ui, inst ) {
var ts, bs, ls, rs, l, r, t, b, i, first,
- inst = $(this).draggable( "instance" ),
o = inst.options,
d = o.snapTolerance,
x1 = ui.offset.left, x2 = x1 + inst.helperProportions.width,
@@ -918,9 +926,9 @@ $.ui.plugin.add("draggable", "snap", {
});
$.ui.plugin.add("draggable", "stack", {
- start: function() {
+ start: function( event, ui, instance ) {
var min,
- o = $(this).draggable( "instance" ).options,
+ o = instance.options,
group = $.makeArray($(o.stack)).sort(function(a,b) {
return (parseInt($(a).css("zIndex"),10) || 0) - (parseInt($(b).css("zIndex"),10) || 0);
});
@@ -936,15 +944,18 @@ $.ui.plugin.add("draggable", "stack", {
});
$.ui.plugin.add("draggable", "zIndex", {
- start: function(event, ui) {
- var t = $(ui.helper), o = $(this).draggable( "instance" ).options;
+ start: function( event, ui, instance ) {
+ var t = $( ui.helper ),
+ o = instance.options;
+
if(t.css("zIndex")) {
o._zIndex = t.css("zIndex");
}
t.css("zIndex", o.zIndex);
},
- stop: function(event, ui) {
- var o = $(this).draggable( "instance" ).options;
+ stop: function( event, ui, instance ) {
+ var o = instance.options;
+
if(o._zIndex) {
$(ui.helper).css("zIndex", o._zIndex);
}