-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(calendar): add day view #275
base: develop
Are you sure you want to change the base?
Conversation
TB-136 Add day view to calendar
While I'm adding the day view to our calendar, I should probably also work on fixing some of the other calendar bugs (e.g. TB-56 and TB-131). |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/tutorbook/tutorbook/3u4URDjPCAwABTt2dGxDGBQnDK3e |
Codecov Report
@@ Coverage Diff @@
## develop #275 +/- ##
===========================================
+ Coverage 67.46% 68.98% +1.52%
===========================================
Files 223 224 +1
Lines 4767 4788 +21
Branches 1277 1288 +11
===========================================
+ Hits 3216 3303 +87
+ Misses 1551 1485 -66
Continue to review full report at Codecov.
|
TB-131 Polish calendar dialog positioning
The calendar meeting dialog should be positioned better so that it's content is always fully on-screen (when the screen height is equal to or greater than the dialog content height, that is). I should try to recreate the Google Calendar dialog positioning algorithm. |
471c64b
to
f65cad8
Compare
f65cad8
to
1fc9a0c
Compare
1fc9a0c
to
5bd445c
Compare
5bd445c
to
629d083
Compare
629d083
to
9958b7d
Compare
9958b7d
to
2c7b2e9
Compare
Fixes TB-136 and TB-131