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

univel.br - see bug description #7247

Closed
jppcel opened this issue Jun 5, 2017 · 6 comments
Closed

univel.br - see bug description #7247

jppcel opened this issue Jun 5, 2017 · 6 comments
Labels
Milestone

Comments

@jppcel
Copy link

jppcel commented Jun 5, 2017

URL: http://univel.br/central-do-aluno/calendario-academico
Browser / Version: Firefox 53.0
Operating System: Windows 10
Problem type: Something else - I'll add details below

Steps to Reproduce

  1. Navigate to: http://univel.br/central-do-aluno/calendario-academico
  2. See the calendar.

Expected Behavior: The calendar of the year.

Actual Behavior: In Firefox, the calendar of the all months are the same of the actual month.
In Chrome the problem not occour.

Screenshot Description

From webcompat.com with ❤️

@adamopenweb
Copy link
Collaborator

adamopenweb commented Jun 5, 2017

Thanks for the report @jppcel! I can reproduce this issue in Firefox 55 and Safari for OSX.

The calendar works as expected in Chrome and Edge.

screen shot 2017-06-05 at 12 29 06 pm

@wisniewskit
Copy link
Member

The reason is given right in the console as a clear warning:

Deprecation warning: value provided is not in a recognized RFC2822 or ISO format.
moment construction falls back to js Date(), which is not reliable across all browsers and versions.
Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release.
Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.

That is, they're passing non-standard date-formats to moment.js, like "1-01-2017" and "2-01-2017" (from their data-month attributes in the HTML). This will simply feed the values into new Date() and that will only work on whatever browser happens to understand that format (which for their strings is Chrome).

So this is technically the site's error. If they change their data-month values to a format that moment.js intrinsically understands, then they will have no problem (such as "2017-01-01" and "2017-02-01").

(Also note that Firefox bug 1274354 is about tracking what compat work to do to resolve this data-parsing mess.)

@karlcow
Copy link
Member

karlcow commented Sep 1, 2017

The people making the web site
https://www.sirius.com.br/

It is possible to contact them at
https://www.sirius.com.br/#contato

@marllonpanisset seems to be working for Sirius Interativa
Maybe he knows people working on univel.br site.

@jppcel
Copy link
Author

jppcel commented Sep 22, 2017

Hello,

I study on Univel. @motoneto works with the website of this university.

@karlcow karlcow added this to the sitewait milestone Oct 30, 2017
@reinhart1010
Copy link

This issue still exists as of today.

@miketaylr miketaylr added the engine-gecko The browser uses the Gecko rendering engine label Apr 30, 2019
@softvision-sergiulogigan

The issue has been fixed.

Tested with:
Browser / Version: Firefox Nightly 70.0a1 (2019-08-15)
Operating System: Windows 10 Pro

image

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

No branches or pull requests

8 participants