Skip to content

feat(calendar): add component propery to fix calendar height #1146

Merged
merged 5 commits into from
Apr 30, 2020

Conversation

MadBrozzeR
Copy link
Contributor

@MadBrozzeR MadBrozzeR commented Apr 28, 2020

Зафиксирована высота календаря (по факту в календаре просто всегда становится шесть недель).
Плюс немного поправлены предупреждения от линтера в рамках дорабатываемого файла.

Мотивация и контекст

Если календарь открывается во всплывающем окне с фиксированной нижней границей, то из-за варьирующейся высоты календаря при перелистывании месяцев с помощью UI стрелок, кнопки постоянно уходят из-под курсора.

@coveralls
Copy link

coveralls commented Apr 28, 2020

Coverage Status

Coverage decreased (-0.006%) to 85.157% when pulling f12aab6 on feature/fixed-calendar-height into dd67787 on master.

@stepancar
Copy link
Contributor

Я за то чтоб опции не было. Потому что миграцию нужно организовывать, кто этим займется - не понятно
Аффекта сильного по сравнению с прошлым поведением вроде нет

@MadBrozzeR
Copy link
Contributor Author

Убрал опцию, обновил упавший снапшот.

@SiebenSieben SiebenSieben merged commit 9d5da61 into master Apr 30, 2020
@delete-merged-branch delete-merged-branch bot deleted the feature/fixed-calendar-height branch April 30, 2020 12:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants