Skip to content
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

The specified value ':)' does not conform to the required format, 'yyyy-MM-dd'. #1412

Closed
JRGould opened this issue Aug 18, 2014 · 20 comments
Closed

Comments

@JRGould
Copy link

JRGould commented Aug 18, 2014

I'm receiving this javascript error in the latest build of chrome canary ( 39.0.2127.0 ) with modernizr 2.8.3 on line 536.

The specified value ':)' does not conform to the required format, 'yyyy-MM-dd'. modernizr.js:536

It looks like the variable smile is set to :) and is being used to set the values on HTML5 form elements that are being generated by modernizr and when the <date> element is created with a value of :) it's causing Canary to fire a validation error.

@ryanseddon
Copy link
Member

@paulirish is this sort of message going to make it to stable dev tools? Seems a bit annoying to do that.

@paulirish
Copy link
Member

Commit: https://codereview.chromium.org/461323006
Bug: https://code.google.com/p/chromium/issues/detail?id=397952

Currently this is on track to go to stable yup. I will talk to the engineers. I don't think this should be Error level messaging.

@paulirish
Copy link
Member

What this looks like:
image

@paulirish
Copy link
Member

Okay, watch this thread: https://code.google.com/p/chromium/issues/detail?id=397952#c3

The webforms engineers are in Tokyo so we should get a response "today". :)

@ryanseddon
Copy link
Member

Sweet thanks @paulirish, we'll keep this open and track the chromium issue.

@chrishough
Copy link

+1

@paulirish
Copy link
Member

thx for reporting @JRGould

@JRGould
Copy link
Author

JRGould commented Aug 19, 2014

Glad I could help! :)

@paulirish
Copy link
Member

Chromium ticket is now here: https://code.google.com/p/chromium/issues/detail?id=405227

Relatedly, Chrome forms team has had to work around Modernizr's feature detects before. :)
http://src.chromium.org/viewvc/blink?view=revision&revision=141086
Sorry about that!

@patrickkettner
Copy link
Member

code is being merged - https://codereview.chromium.org/491703002

Its going to be updated/reverted in Chrome 38, which will be beta soon. Discussion is in the thread, but they are basically only showing the error when the input is visible

@paulirish
Copy link
Member

​Slightly more specific:

  • Chrome 38 is going to beta soon and will not have this error at all.
    The change will be completely reverted there tanks to the merge on the
    chromium ticket listed above.
  • The codereview link features a patch that will be landing in Chrome 39
    (Canary now). It adjusts the error to be 1) a warning and 2) not triggered
    on Modernizr. Somehow. Magic mostly.

@heby
Copy link

heby commented Aug 21, 2014

image
Google Chrome 38.0.2125.0 dev-m (64-bit)

@patrickkettner
Copy link
Member

@heby - to be clear, this change is happening between the versions of 38 that ships in dev and the version that ships to beta.

@L422Y
Copy link

L422Y commented Aug 22, 2014

:)

@isarmstrong
Copy link

What did I do before the days of rapid-indexing on Google? I would have spent hours trying to figure out what I'd broken.

@RobinQu
Copy link

RobinQu commented Aug 25, 2014

+1

@patrickkettner
Copy link
Member

This was fixed in chrome

@snymax
Copy link

snymax commented Jan 21, 2015

I am getting this error in node-webkit am I stuck with it until nw.js is updated

@patrickkettner
Copy link
Member

@snymax this isn't an error - its just a super annoying warning. you can ignore it.

@patrickkettner
Copy link
Member

@brilyuhns what do you mean?

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

No branches or pull requests

10 participants