fix(calendar): fix minDate/maxDate #1543
Conversation
fulcanellee
requested review from
reme3d2y,
SiebenSieben and
hextion
as code owners
🦋 Changeset detectedLatest commit: db83588 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
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 |
|
Собрана новая демка. |
|
|
||
| // отсекаем лишние месяцы если задана минимальная дата | ||
| if (minDate) { | ||
| generatedMonthsWithWeeks = generatedMonthsWithWeeks.filter( |
There was a problem hiding this comment.
почему решил это делать отдельными проходами, а не в основном цикле?
There was a problem hiding this comment.
Решил что так будет более последовательно. В основном цикле мы можем отфильтровать только prevYearMonths и nextYearMonths (currYearMonths придется фильтровать вне цикла). Причем каждый из трёх отрезков придется проверять как на minDate так и на maxDate.
There was a problem hiding this comment.
А их много там получается? Кажется это можно заменить на один reduce
Исправлено количество отображаемых месяцев при использовании minDate/maxDate пропсов
https://jira.moscow.alfaintra.net/browse/DS-7885