Akhil/mobile marketing sweep - #231
Merged
Merged
Conversation
Add @media (max-width: 640px) rule to input, textarea, and select elements to prevent iOS Safari from auto-zooming on focus. Inputs with font-size < 16px trigger the zoom; setting 16px+ disables this behavior. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…rt drive per-scheme color
- Add pt-[env(safe-area-inset-top)] to fixed header to respect device notch - Update help page to account for safe-area inset in addition to header height (pt-16 md:pt-8 → pt-[calc(theme(spacing.16)+env(safe-area-inset-top))] md:pt-[calc(theme(spacing.8)+env(safe-area-inset-top))]) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Raises the header hamburger button from h-10 w-10 (40px) to h-11 w-11 (44px) to meet the WCAG 2.5.5 minimum touch-target size on mobile. Mobile-nav bottom-bar items were already ≥44px via navItemStyles h-12 w-14 and required no changes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…utilities Final-review finding: bare 'font-size: 16px' on element selectors loses specificity to any Tailwind utility class (text-sm, text-xs, etc.) used on input/textarea/select. Adds !important so the rule defeats utility overrides — standard pattern for this iOS Safari workaround.
…devices bottom-6 (24px) was rendering partially under the iPhone home indicator. Switched to bottom-[max(1.5rem,calc(env(safe-area-inset-bottom)+0.5rem))] so the pill clears the inset on notched devices while keeping the same 24px offset on flat-bezel devices.
…tools on mobile Adds <MobileDesktopHint /> banner that appears only on mobile viewports for 16 tool routes whose multi-pane editor UIs are cramped on phones (api-client, sql-client, database-explorer, redis-commander, csv-excel-json, encryption-playground, s3-drive, environment-manager, notes, to-do, bookmarks, break-room, api-keys, image-compressor, image-to-base64, svg-optimizer). Dismissed state persists per tool via sessionStorage. Mounted once in ClientLayout above the scroll area — no per-tool edits. Per-tool mobile layout fixes deferred to follow-up sub-projects.
…deck splash color The 'random icon' users saw on mobile home screens was the white glyph icon (logo-light.png clone) rendered on iOS/Android's default light launcher background — visually invisible. Replaces icon-192x192.png and icon-512x512.png with the dark glyph (logo-dark.png) so the app icon is visible on light home screens. Manifest background_color flipped from #ffffff to #0a0b12 so the PWA splash matches the in-app dark deck theme instead of flashing white before first paint. Resolves Task 1 of the mobile-foundations sub-project (deferred 'random icon' report) — root cause was the PWA install icon, not the in-app header logo (which was already correct).
Mobile dashboard users had no access to the workspace switcher or notifications bell because both lived inside NavBar (hidden md:block) and the trigger buttons themselves were hidden md:inline-flex. - Drop the hidden md:inline-flex gate on both trigger buttons so they render wherever they are mounted (desktop NavBar is still hidden via its own md:block parent — visibility unchanged). - Add a mobile-only top strip in ClientLayout (md:hidden, with safe-area-inset-top padding) that mounts WorkspaceSwitcher on the left and NotificationsBell on the right. Resolves the sub-project #4.5 NavBar mobile gap flagged during the dashboard sweep.
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Settings page hero kicker was a static 'Workspace' label with no context. Mobile users had no other surface showing which org/workspace they were in while editing settings. Replaces the static label with the active org name + workspace name when both are loaded, falling back to the old label otherwise. Subscribes to useActiveOrg/useActiveWorkspace selectors so switching workspace via the dashboard top strip re-renders the kicker immediately.
Mirrors the settings page fix. Profile hero kicker now shows <Briefcase> Org / Workspace from Zustand selectors when both load, falling back to 'Developer identity' label otherwise. Reactive to workspace switcher anywhere.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.