Datepicker: Treat all hidden text fields the same

[severin]

File : jquery.ui.datepicker.js In function _findPos (line 755)

"jQuery.expr.filters.hidden(obj)" must be added to test hidden text inputs.

while (obj && (obj.type == 'hidden' + while (obj && (obj.type == 'hidden'

obj.nodeType != 1)) {
obj.nodeType != 1	jQuery.expr.filters.hidden(obj))) {

[severin]

while (obj && (obj.type == 'hidden' || obj.nodeType != 1 **|| jQuery.expr.filters.hidden(obj)**)) {

[Scott González]

Can you explain the use case for showing a datepicker that is tied to an element that's hidden?

[severin]

HTML :

<input type="text" name="date" class="datepicker" />

CSS :

.datepicker{
    display:none;
}

JS :

$('.datepicker').datepicker({showOn:'button'});

[severin]

The purpose of this is to display only the button (not the input).

But it's only a design hidden. For most of the views using this componant, the input is displayed, the designer wants to hide it only on one view and he only has CSS to do so.

What happen today : the datepicker pop in the top left corner.

[Scott González]

Datepicker: Properly handle datepickers attached to inputs that are hidden using CSS. Fixes #6988 - Datepicker: Treat all hidden text fields the same. (cherry picked from commit 6fbee255d8d895c7e84a4b2eaad2cfadc8a3425a)

Changeset: cf9dac6c1704150d40191573767e5bfa58dcd9d2

[Scott González]

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

Changeset: a3cba34a4d99756e7590fee391fee1bdf06d8a5f

[mikesherov]

#5658 is a duplicate of this ticket.