-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Button does not remove disabled on page refresh #793
Comments
can you provide a jsfiddle? |
How is that possible that button states survive page refresh? |
Try this: |
That's clearly a Firefox sorcery :P This does not happen in Chromium/Opera. |
Calling $('.btn').button('reset') on load does the trick... Looks like the browser is keeping the disabled attribute. |
Looks like this is a "feature": http://stackoverflow.com/questions/5985839/bug-with-firefox-disabled-attribute-of-input-not-resetting-when-refreshing Seems setting |
wow that's crazy, but not something I think we should deal with in bootstrap -- what do you guys think? |
I think there's no point making a "fix" just for Firefox. @cgunther and |
It should at least be documented if no fix is to be applied. |
Rather than resetting the button states on load, what if we reset the button state on unload of the page? |
just added a note here - 6e8a3bb don't think we want to do the unload incase someone is relying on that for something (as weird as that would be) |
Excuse me, but if you have a submit button that remembers a disabled state and think just because it happens in Firefox that you don't need a fix, you're fooling yourself. Many people use Firefox and if you have a commercial or donation site, having an active donate button on refresh after people might have errors in their form is EXTREMELY important. Yeah, autocomplete="off" apparently works, but autocomplete="off" should have been the default. |
I thought the whole point of frameworks like bootstrap or jquery is that they insulate you from the peculiarities of different browsers. |
This issue seems out of scope of Bootstrap's purpose as primarily a CSS framework to fix this. Maybe Firefox is doing it "right" so if it is "fixed" perhaps the fix should be to make the other browsers like Firefox. (playing devil's advocate here) Finally, how do you actually fix it? On page load a script could look for all disabled buttons and enable them, but how would that script know that those buttons shouldn't be disabled? The only solution seems to be to re-enable them on page unload. -- edit: Just realized this is an old post but the issue seems to still exist as of FF23: |
Ran into the same weird bug today, I think it should be included in Bootstrap for the sole reason that the framework should give a uniform behavior in all browsers, now I still have to apply browser specific fixes! |
Another "fix" for this FF bug is to change from using a |
Also, link to the Mozilla bug in the relevant callout.
Add Wall of Browser bugs entry for #793
Also, link to the Mozilla bug in the relevant callout.
When the state of a button is changed there is an error if the page is refreshed without resetting the button.
This has happened when I have been in development an AJAX call has failed for whatever reason and the button state not reset. The page gets refreshed with the latest code changes but the button still has a 'disabled' attribute. A solution would be calling something like this when loading the page:
But would be better if the script did this automatically to have buttons in their correct state at page load.
The text was updated successfully, but these errors were encountered: