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

DateRangePicker: Clicking on date selects the previous day for some timezones #2182

Closed
Neeeko opened this issue Sep 4, 2020 · 0 comments · Fixed by #2203
Closed

DateRangePicker: Clicking on date selects the previous day for some timezones #2182

Neeeko opened this issue Sep 4, 2020 · 0 comments · Fixed by #2203
Assignees
Labels
approved This issue is approved bug This issue is a bug in the code Medium Prio TOPIC B
Milestone

Comments

@Neeeko
Copy link

Neeeko commented Sep 4, 2020

Describe the bug
DateRangePicker - On some timezones, when clicking on a date, the previous day is selected. For example, clicking on April 9th => April 8th is displayed.
The DatePicker works properly though, only the DateRangePicker has this issue.

To reproduce
Steps to reproduce the behavior:

  1. Change your computer timezone to a timezone in the US West Coast (for example in Anchorage, Alaska). (Using a Mac, go to Preferences => Date and Time => Time zone.
  2. Go to https://sap.github.io/ui5-webcomponents/playground/components/DateRangePicker/ and open the Date Range Picker.
  3. Select a date (For example September 4th). You'll see that September 3rd has been selected.

Expected behavior
The selected dates should be those that have been clicked on.

Screenshots
Screen Recording 2020-09-03 at 8 40 55 AM
This gif was recorded by a colleague in California (California Timezone).

Context

  • UI5 Web Components version: 0.23.1
  • OS/Platform: macOS Catalina

Affected components (if known)
ui5-daterange-picker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This issue is approved bug This issue is a bug in the code Medium Prio TOPIC B
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants