Skip to content

fix: hide bookings opt-in banner on mobile viewport#27965

Merged
dhairyashiil merged 1 commit intomainfrom
devin/1771164922-hide-bookings-banner-mobile
Feb 15, 2026
Merged

fix: hide bookings opt-in banner on mobile viewport#27965
dhairyashiil merged 1 commit intomainfrom
devin/1771164922-hide-bookings-banner-mobile

Conversation

@PeerRich
Copy link
Copy Markdown
Member

What does this PR do?

Hides the "Try the new bookings page" opt-in banner on mobile viewports (< 768px) by adding hidden md:block Tailwind classes to the banner container.

The banner is a fixed-position popup in the bottom-right corner of /bookings which takes up significant screen space on mobile devices.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A — CSS-only visibility change.

How should this be tested?

  1. Navigate to /bookings on a desktop viewport — the opt-in banner should still appear in the bottom-right corner
  2. Resize the browser below 768px (or use mobile device emulation) — the banner should not be visible
  3. Verify the banner's "Try it" button and dismiss functionality still work on desktop

Human Review Checklist

  • Confirm md (768px) is the desired mobile breakpoint — adjust to sm (640px) or lg (1024px) if a different threshold is preferred
  • Note: the banner hook/state logic still runs on mobile; only CSS visibility is toggled. This is an acceptable tradeoff for a one-line fix.

Co-Authored-By: peer@cal.com <peer@cal.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@dhairyashiil dhairyashiil marked this pull request as ready for review February 15, 2026 14:17
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Copy link
Copy Markdown
Member

@dhairyashiil dhairyashiil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

working 👍🏼

Screen.Recording.2026-02-15.at.7.52.35.PM.mov

@dhairyashiil dhairyashiil enabled auto-merge (squash) February 15, 2026 14:23
@dhairyashiil dhairyashiil merged commit 773fab8 into main Feb 15, 2026
51 of 52 checks passed
@dhairyashiil dhairyashiil deleted the devin/1771164922-hide-bookings-banner-mobile branch February 15, 2026 14:24
emrysal pushed a commit that referenced this pull request Feb 18, 2026
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants