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

Year range in localized Datepicker is not correct #4674

Closed
saeedtabrizi opened this issue Jul 11, 2016 · 6 comments
Closed

Year range in localized Datepicker is not correct #4674

saeedtabrizi opened this issue Jul 11, 2016 · 6 comments
Labels
duplicate This issue or pull request already exists

Comments

@saeedtabrizi
Copy link

Year Range in Localized Datepicker is not correct . while you set a localized calendar like the persian calendar just you must see year button for localized range of years .

it seems in CalendarYear.js file and getYears method the min and max value for year range not localized .

Steps to reproduce

1.declare const dtf_fa = Intl.DateTimeFormat;
2. in render method <DatePicker hintText="Portrait Dialog" DateTimeFormat={dtf_fa} locale="fa-IR-u-ca-persian" />
3. run and see year button value . its shows different number .(in my case show 2016 that equals to localized persian calendar 1395) .

Versions

  • Material-UI: 0.15.2
  • React: 15.2
  • Browser: Chrome 51

dddddd

@manishoo
Copy link

I have this problem too ,the day is not correct either ! e.g it says 22 tir while on the calendar 12 tir is active.

@mbrookes
Copy link
Member

@alitaheri Any chance you could comment on this? Thanks.

@alitaheri
Copy link
Member

I haven't used this component for my application nor have I used any other date pickers for this purpose. It would be great if this component could be tweaked to support Persian calendar system. I don’t really know how to do it though, I never touched this component, sorry folks 😅 😅

@saeedtabrizi
Copy link
Author

hi @mehdijjz
i made a patch for solve this issue . i'll send it as a PR asap .

@mpontikes mpontikes mentioned this issue Aug 5, 2016
13 tasks
@aahan96
Copy link
Contributor

aahan96 commented Aug 17, 2016

@saeedtabrizi Any update on the PR?

@oliviertassinari oliviertassinari added duplicate This issue or pull request already exists bug 🐛 Something doesn't work labels Sep 15, 2016
@oliviertassinari
Copy link
Member

I'm closing as I think it's a duplicate of #5101.

@oliviertassinari oliviertassinari removed the bug 🐛 Something doesn't work label Feb 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

6 participants