This project is archived and is in readonly mode.
data-confirm and data-method don't work together in Internet Explorer 8
-
Jeff Kreeftmeijer
Automatic cleanup of spam.
-
Yuval Kordov
I notice the jquery UJS is still using the following for confirms:
$('body').delegate('a[data-confirm], button[data-confirm], input[data-confirm]', 'click.rails', function () {This was a response to a bug in JQuery 1.4.3 which has since been resolved in 1.4.4. It should be reverted to:
$('a[data-confirm],input[data-confirm]').live('click', function(e) { -
Neeraj Singh
@Yuval Kordov : The change that you are proposing fixes the problem? Or was it just a comment in general.
Current version of jquery-ujs supports both jQuery 1.4.4 and jQuery 1.4.3 . We can drop the support for jQuery 1.4.3 if that change fixes the problem.
-
Utkarsh Kukreti
- Tag set to fixed
This was fixed in Prototype UJS, https://github.com/rails/prototype-ujs/issues/closed#issue/3
-
Neeraj Singh
Can someone confirm if this is an issue in jquery-ujs or not?
-
Utkarsh Kukreti
It works fine on jquery-ujs.
-
Yuval Kordov
Neeraj, I presented the delegate fix due to the JQuery 1.4.3 bug. When we upgraded to 1.4.4, the delegate call no longer worked, so we reverteds to the .live call with no further issue. Personally, I would suggest dropping support for 1.4.3 but I've obviously only tested within my own framework. Your mileage may vary.
-
Santiago Pastorino
- State changed from new to resolved