-
Notifications
You must be signed in to change notification settings - Fork 185
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
fix: enlarge calendar day hover area #7750
fix: enlarge calendar day hover area #7750
Conversation
…y-area # Conflicts: # packages/vkui/src/components/CalendarDay/CalendarDay.tsx
size-limit report 📦
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
e2e tests |
👀 Docs deployed
Commit 0c5e281 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7750 +/- ##
=======================================
Coverage 95.14% 95.14%
=======================================
Files 384 384
Lines 11354 11357 +3
Branches 3724 3727 +3
=======================================
+ Hits 10803 10806 +3
Misses 551 551
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
А почему скриншоты поменялись? |
В UPD отписал
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не ну гений
Так там не столько размер поменялся, как цвет? Или мне кажется? |
Не, цвета такие же, проверил, через сайт для сравнения двух картинок |
Действительно странно, возможно проблема в том, что я ветку создал от #7738. Главное, что при сравнении с мастером цвета не изменились |
✅ v7.0.0-beta.1 🎉 |
✅ v7.0.0 🎉 |
Описание
Сейчас при выборе range в
CalendarRange
выбранный промежуток промаргивает(см в приложенном видео). Происходит это из-за промежутка между линиями в месяце. При попадании мыши туда, выбранный промежуток сбрасывается из-за чего происходит промаргивание. Нужно придумать как от этого избавитьсяИзменения
Убрал промежуток между неделями в месяце. Визуально сделал промежуток засчет вертикальных паддингов у ячеек дня. Засчет этого визуально ничего не поменялось, но промаргивания пропали.
ДО
default.mov
ПОСЛЕ
default.mov
UPD
Скриншоты изменились для
Calendar
, так как для скрытых дней при размереs
ранее выставлялся неправильный класснейм и соответственно размер.Release notes
Исправления