From 9e4205ddfde3077913f7ded01bbe903c58409be9 Mon Sep 17 00:00:00 2001
From: pzgz <leon.guan@gmail.com>
Date: Tue, 11 Jun 2013 16:33:30 +0800
Subject: [PATCH] Use attr to set disabled attribute for `a` tag, instead of
 using prop

Since anchor tag `a` doesn't actually support `disabled` attribute. So,
if we use `prop` to disable a link, the css selector `a[disabled]`
won't work. In some css frameworks, such as bootstrap, the anchor won't
have any differences. Using attr() will make things different.
---
 src/rails.js | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/src/rails.js b/src/rails.js
index 50121d6e..4cf926d4 100644
--- a/src/rails.js
+++ b/src/rails.js
@@ -36,10 +36,10 @@
     formInputClickSelector: 'form input[type=submit], form input[type=image], form button[type=submit], form button:not([type])',
 
     // Form input elements disabled during form submission
-    disableSelector: 'input[data-disable-with], button[data-disable-with], textarea[data-disable-with]',
+    disableSelector: 'input[data-disable-with], button[data-disable-with], textarea[data-disable-with], a[data-disable-with]',
 
     // Form input elements re-enabled after form submission
-    enableSelector: 'input[data-disable-with]:disabled, button[data-disable-with]:disabled, textarea[data-disable-with]:disabled',
+    enableSelector: 'input[data-disable-with]:disabled, button[data-disable-with]:disabled, textarea[data-disable-with]:disabled, a[data-disable-with]:disabled',
 
     // Form required input elements
     requiredInputSelector: 'input[name][required]:not([disabled]),textarea[name][required]:not([disabled])',
@@ -182,10 +182,15 @@
     */
     disableFormElements: function(form) {
       form.find(rails.disableSelector).each(function() {
-        var element = $(this), method = element.is('button') ? 'html' : 'val';
+        var element = $(this), method = (element.is('button') || element.is('a')) ? 'html' : 'val';
         element.data('ujs:enable-with', element[method]());
         element[method](element.data('disable-with'));
         element.prop('disabled', true);
+        if(element.is('a')) {
+          element.attr('disabled', true);
+        } else {
+          element.prop('disabled', true);
+        }
       });
     },
 
@@ -198,6 +203,11 @@
         var element = $(this), method = element.is('button') ? 'html' : 'val';
         if (element.data('ujs:enable-with')) element[method](element.data('ujs:enable-with'));
         element.prop('disabled', false);
+        if(element.is('a')) {
+          element.removeAttr('disabled');
+        } else {
+          element.prop('disabled', false);
+        }
       });
     },
 
@@ -263,6 +273,7 @@
     disableElement: function(element) {
       element.data('ujs:enable-with', element.html()); // store enabled state
       element.html(element.data('disable-with')); // set to disabled state
+      element.attr('disabled', true);
       element.bind('click.railsDisable', function(e) { // prevent further clicking
         return rails.stopEverything(e);
       });
@@ -274,6 +285,7 @@
         element.html(element.data('ujs:enable-with')); // set to old enabled state
         element.removeData('ujs:enable-with'); // clean up cache
       }
+      element.removeAttr('disabled');
       element.unbind('click.railsDisable'); // enable element
     }
 
@@ -390,4 +402,4 @@
     });
   }
 
-})( jQuery );
+})( jQuery );
\ No newline at end of file