Skip to content
This repository has been archived by the owner on Feb 7, 2019. It is now read-only.

DatePicker crashes on Firefox #25

Closed
ShortlyMAB opened this issue Aug 16, 2013 · 11 comments
Closed

DatePicker crashes on Firefox #25

ShortlyMAB opened this issue Aug 16, 2013 · 11 comments
Labels

Comments

@ShortlyMAB
Copy link

"http://mozilla.github.io/brick/dist/brick.js:2" I got this crash when opening DatePicker demo page everytime with Firefox.

@oksome
Copy link

oksome commented Aug 16, 2013

Same error here, for both x-calendar and x-datepicker demos :

Tested on Ubuntu 13.04 x86_64, Firefox 23 (Ubuntu) and Firefox Nightly UX 26.0a1 (2013-08-15)

@hutchy2570
Copy link

It also crashes in Chrome on the calendar & datepicker demos.

Chrome 28.0.1500.95 on OSX 10.6.8

@ldoubleuz
Copy link
Contributor

Just for reference, what regions are your browsers running in? I have a hunch that it may be an issue with different dateformats.

Also, can you tell me if you are seeing the native browser datepicker, or the polyfilled text input version?

@csuwildcat
Copy link
Contributor

@aydnbhdr we've collectively run all recent and edge versions of Firefox on Windows, Linux, and OS X, and still haven't been able to replicate this. Definitely seems to be either a regional or JS Date issue IMO, possible something like this --> uxsolutions/bootstrap-datepicker#95 (comment)

@fwenzel
Copy link

fwenzel commented Aug 16, 2013

@sole was able to reproduce I think. Sole, are you running a Spanish browser?

@sole
Copy link
Contributor

sole commented Aug 16, 2013

Yes it's reproducible ;-)
My computer is set to use UK region settings

screen shot 2013-08-16 at 17 04 35
screen shot 2013-08-16 at 17 05 43
screen shot 2013-08-16 at 18 28 01

@hutchy2570
Copy link

I'm set to UK too.

@ldoubleuz
Copy link
Contributor

For now, we've pushed a stopgap fix as a 1.0.2beta release to prevent the infinite looping causing the crash. However, we are still working on reproducing/debugging the underlying issue caused by non-US dateformats.

@fwenzel
Copy link

fwenzel commented Aug 17, 2013

What commit is that fix? Thanks

@ldoubleuz
Copy link
Contributor

7ebc9fd

@ldoubleuz
Copy link
Contributor

Fixed as of version 1.0.3beta

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

7 participants