fix(universal-date-input): fix date validation #1545
Conversation
fulcanellee
requested review from
reme3d2y,
SiebenSieben and
hextion
as code owners
🦋 Changeset detectedLatest commit: 11df06b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Собрана новая демка. |
| end: max, | ||
| }); | ||
| const date = clamp( | ||
| new Date(`${segments.year}-${segments.month}-${segments.day}T00:00:00`), |
There was a problem hiding this comment.
тут просто сбросил время на 00:00:00?
There was a problem hiding this comment.
Добавил временную метку, так как без нее new Date в JS автоматически преобразует её в локальное время пользователя, основываясь на его часовом поясе (это ощутимо при смене часовых поясов с очень большой разницей, например добавляем utc 06.01, а в Гонолулу будет 05.01, что приводит к ошибке которую зарепортили).
При добавлении T00:00:00, мы указаываем время в дополнение к дате. JS интерпретирует такую строку как дату и время локальной временной зоны
There was a problem hiding this comment.
Какой вариант хотелось получить?
Я к тому, что без Z - это все равно будет локально время. Или так и нужно было?
There was a problem hiding this comment.
Так и нужно было. Без Z будет локальное время, и оно не будет смещаться при смене часового пояса.
На скрине в первом случае одна дата. Во втором, ввели 5ое число, а получили 4ое
Исправление валидации вводимого времени
https://jira.moscow.alfaintra.net/browse/DS-8066