This is a great little module. But like many great pieces of code it works greater in Firefox and Webkit browsers and Internet Explorer spoils the party. I'm using a page view (Views 2) and the multi-select boxes don't work properly with Internet Explorer. I'm using a filter with one vocabulary with three terms. In IE, I can select a checkbox and nothing happens. Or I will select or de-select a box and get the wrong response.

Works great in Chrome and Firefox.

CommentFileSizeAuthor
#6 viewslivefilters_ie8.patch545 bytesjohn.money
#5 888446-5-viewslivefilter.patch544 bytesalberto56

Comments

mediamash’s picture

mediamash commented

when clicking on the page after selecting it works
seems to be a bug

buntstich’s picture

buntstich commented

#subscribe

Dropdown menu works fine. For checkboxes you need a second click on page background.

bisscomm’s picture

bisscomm commented

Just solved the problem in very simple way.

Change the lines (in viewslivefilters.js):

// Submit form immediately when a form element's value changes.
$('.views-exposed-widget .form-item :input', form).bind('change', function() {
throb(this);
$(form).submit();
});

for

// Submit form immediately when a form element's value changes.
$('.views-exposed-widget .form-item :input', form).bind('click', function() {
throb(this);
$(form).submit();
});

john.money’s picture

john.money commented

#3 fixes issue... patch needed, dev update even better.

alberto56’s picture

alberto56 commented
Title: Internet Explorer 8 » Internet Explorer 8 and 7 (ie7, ie8)
Status: Active » Needs review
StatusFileSize
new 888446-5-viewslivefilter.patch544 bytes

I agree this is pretty critical because of the IE8 and IE7 market share. Here's a patch using bisscomm's work, above. Thanks.

This works in IE7, IE8, Chrome, Safari, FF.

Haven't had a chance to test this with Chrome, though...

john.money’s picture

john.money commented
StatusFileSize
new viewslivefilters_ie8.patch545 bytes

Not so fast... #3 fixes IE issues with non-text elements, but also renders text field elements unusable on all browsers (submit on click).

Better selector is:

$('.views-exposed-widget .form-item :input:not(.form-text)', form).bind('click', function() {