Datepicker: Properly handle datepickers attached to inputs that are hidden using CSS. Fixes #6988 - Datepicker: Treat all hidden text fields the same.

scottgonzalez · scottgonzalez · commit a3cba34a4d99 · 2011-02-15T08:39:47.000-05:00
diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js
@@ -754,7 +754,7 @@ $.extend(Datepicker.prototype, {
 	_findPos: function(obj) {
 		var inst = this._getInst(obj);
 		var isRTL = this._get(inst, 'isRTL');
-        while (obj && (obj.type == 'hidden' || obj.nodeType != 1)) {
+        while (obj && (obj.type == 'hidden' || obj.nodeType != 1 || $.expr.filters.hidden(obj))) {
             obj = obj[isRTL ? 'previousSibling' : 'nextSibling'];
         }
         var position = $(obj).offset();

Original file line number	Diff line number	Diff line change
`@@ -754,7 +754,7 @@ $.extend(Datepicker.prototype, {`
`754`	`754`	`_findPos: function(obj) {`
`755`	`755`	`var inst = this._getInst(obj);`
`756`	`756`	`var isRTL = this._get(inst, 'isRTL');`
`757`		`- while (obj && (obj.type == 'hidden' \|\| obj.nodeType != 1)) {`
	`757`	`+ while (obj && (obj.type == 'hidden' \|\| obj.nodeType != 1 \|\| $.expr.filters.hidden(obj))) {`
`758`	`758`	`obj = obj[isRTL ? 'previousSibling' : 'nextSibling'];`
`759`	`759`	`}`
`760`	`760`	`var position = $(obj).offset();`