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

Dated Athlete Settings effective date changes without explanation #705

Closed
dlh3 opened this issue Nov 27, 2018 · 4 comments
Closed

Dated Athlete Settings effective date changes without explanation #705

dlh3 opened this issue Nov 27, 2018 · 4 comments

Comments

@dlh3
Copy link
Contributor

dlh3 commented Nov 27, 2018

Step 1: Are you in the right place?

Yes.

Step 2: Describe your environment

  • Plugin version: 6.8.0 Stable
  • Chrome/Opera/Chromium version: Chrome 70.0.3538.102
  • OS version: MacOS 10.14

Step 3: Describe the problem:

Steps to reproduce:

  1. Open Athlete Settings
  2. Add new dated athlete settings
  3. Set the "Athlete settings since" date (either by typing the date, or with date picker)
  4. Click out of the "Athlete settings since" field

Actual Behavior:

  • The date changes to the day before (if you entered 11/11/2018, it becomes 11/10/2018)

Expected Behaviour:

  • I suppose this is the expected behaviour, but that isn't obvious. Perhaps a note add to that field would reduce confusion.

Media attachment:

None.

Providing your Elevate synced athlete activities can be useful to solve the problem

Not related to specific activities.

@thomaschampagne
Copy link
Owner

@dlh3 I can't reproduce your problem :/ Pb still there?

@dlh3
Copy link
Contributor Author

dlh3 commented Dec 17, 2018

@thomaschampagne Here's a screen recording to demonstrate the reported behaviour. First I type a date (11/19/2018) into the "Athlete settings since" field, then when I click out, the value moves to the previous day (11/18/2018). Next I use the date picker to select a date (11/21/2018), but when the date picker closes, the "Athlete settings since" field is the previous day (11/20/2018).

ezgif com-video-to-gif
(sorry for the small size, you may need to zoom in)

Once again, I am guessing this is the intended behaviour. If so, that is fine, but I think adding an explanation note to the field could reduce confusion so other people don't think it is a bug (as I did).

@dlh3
Copy link
Contributor Author

dlh3 commented Dec 30, 2018

@thomaschampagne thanks to @cwparsons, I think I know why you couldn't reproduce this! Timezone difference.

I just tested this and confirmed that the date would consistently change when my system is set to UTC-8, but the behaviour stops when I set my system to UTC+1.

It seems that the date field and picker function correctly from UTC and ahead (east), but this date shifting behaviour occurs for any timezones behind (west of) UTC.

@dlh3
Copy link
Contributor Author

dlh3 commented Jan 2, 2019

Looks like this is an OOTB issue with Angular-Material, see angular/components#7167.

One user contributed a custom solution: angular/components#7167 (comment)

But the MatMomentDateModule may be the best solution: angular/components#7167 (comment)

thomaschampagne added a commit that referenced this issue Jan 12, 2019
Add MomentDateAdapter to solve shifting dates issue; fixes #705
pull bot pushed a commit to dongbum/elevate that referenced this issue Oct 17, 2020
pull bot pushed a commit to dongbum/elevate that referenced this issue Oct 17, 2020
Add MomentDateAdapter to solve shifting dates issue; fixes thomaschampagne#705
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

2 participants