-
Notifications
You must be signed in to change notification settings - Fork 10.2k
feat: Add tablet layout to Booker #20742
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
base: main
Are you sure you want to change the base?
Conversation
@evan-liu is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
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.
mrge found 1 issue across 7 files. View it in mrge.io
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (04/17/25)1 reviewer was added to this PR based on Keith Williams's automation. "Add community label" took an action on this PR • (04/17/25)1 label was added to this PR based on Keith Williams's automation. |
589a42e
to
2b25134
Compare
// Same as month_view | ||
tablet: false, |
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.
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.
I don't think it's looks good now. cc @PeerRich
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.
Option B: Move meta to the left side:
(resize the viewport to be 768px - 1024px
, click "back" and time slots to see the transition)
Option C: In dialog, same as the mobile layout.
Please let me know which option is preferred or if you have any other ideas for trying out.
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.
oh yeah no bueno. lemme ask @ciaranha
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.
I like Option C: Modal - View in Figma
https://github.com/user-attachments/assets/b932bfba-852c-441d-b3a1-eafe8aa2d15d
I've also done designs here for:
- Option B: Move the left side View in Figma
- Option C: single column option View in Figma
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.
Thanks @ciaranha.
- I had assumed it was full screen before — I’ll switch to fixed widths.
- Just to confirm: the meta section should be 1 column (unlike the 2 columns in [CAL-5253] booking page: new tablet view for #19698), correct?

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.
Hi @ciaranha @anikdhabal, I've changed to the modal option, please have a look:
(resize the viewport to be 768px - 1024px
, click time slots to open the booking modal)
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.
This is a great PR btw
please know that this might take some time as it requires approval from both platform and foundation team 🙏 |
a60f915
to
5957ea3
Compare
5957ea3
to
6994d4f
Compare
This PR is being marked as stale due to inactivity. |
Warning Rate limit exceeded@anikdhabal has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 37 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (3)
✨ Finishing Touches
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Summary by mrge
Added tablet layout to the Booker component for improved user experience on medium-sized screens (768px-1024px viewport width).
UI Improvements
What does this PR do?
Booking page: new tablet view
Visual Demo (For contributors especially)
A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).
Live Preview:
(resize the viewport to be
768px - 1024px
, click time slots to open the booking form)Mandatory Tasks (DO NOT REMOVE)