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

Re-open #4803: moment().format() incorrect on older browsers #4972

Closed
Unsfer opened this issue Jan 28, 2019 · 6 comments
Closed

Re-open #4803: moment().format() incorrect on older browsers #4972

Unsfer opened this issue Jan 28, 2019 · 6 comments

Comments

@Unsfer
Copy link

Unsfer commented Jan 28, 2019

re-open #4803
Tried on the latest (2.24.0) version, still the same result

var date = "1983-04-01";
var ruDate = moment(date, 'YYYY-MM-DD').format('DD.MM.YYYY');
alert(date + " - " + ruDate);
// Output: "1983-04-01 - 31.03.1983"
@ashsearle
Copy link
Contributor

@Unsfer What browser(s) are you testing in these days?

If these issues are due to bugs in the browse's implementation of Date, it won't be resolved in moment 2.x (see conversation in issue 4469)

@Unsfer
Copy link
Author

Unsfer commented Jan 28, 2019

Running on Chromium 35 and Firefox 45.9.0
And sadly our env tied on old browsers

@Unsfer
Copy link
Author

Unsfer commented May 12, 2020

@marwahaha Is it just abandoned?

Tested on last available version, still the same result

https://jsfiddle.net/jv9tzmxr/

var date = "1983-04-01";
var ruDate = moment(date, "YYYY-MM-DD").format("DD.MM.YYYY");
alert("Moment version: " + moment.version + ". \nTest: " + date + " - " + ruDate);
// Output:
// Moment version: 2.25.3. 
// Test: 1983-04-01 - 31.03.1983

@marwahaha
Copy link
Member

I cannot reproduce this:
image

What time zone or locale are you in?

@Unsfer
Copy link
Author

Unsfer commented May 12, 2020

The bug only reproduced on browsers mentioned above.

Locale/timezone: ru +0400

The reason probably #4803 (comment)

@marwahaha marwahaha reopened this May 12, 2020
@marwahaha
Copy link
Member

As ashsearle mentioned, if they are related to browser implementation of Date, they cannot be fixed by Moment 2.x

Sorry you are experiencing this issue.
https://momentjs.com/docs/#/-project-status/

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

3 participants