Skip to content

fix: filter sheet modal, form card width, and timezone label overflow on mobile#410

Merged
gaghan430 merged 1 commit intomainfrom
fix/mobile-filter-form-timezone
Apr 17, 2026
Merged

fix: filter sheet modal, form card width, and timezone label overflow on mobile#410
gaghan430 merged 1 commit intomainfrom
fix/mobile-filter-form-timezone

Conversation

@gaghan430
Copy link
Copy Markdown
Contributor

Summary

Follow-up fixes from the responsive mobile improvements (#409).

  • Filter sheet: set modal={false} on the DataTableToolbar Sheet so portaled Radix dropdowns (SelectFilter options) receive clicks instead of being blocked by the modal overlay
  • Form card width: Col span={12} offset={6}Col xs={24} md={{ span: 12, offset: 6 }} so form cards use full width on mobile (5 pages: contact create/detail, contact-group create/detail, profile settings)
  • Timezone select: wrap label in truncate span to prevent long timezone labels from overlapping the chevron icon

Test plan

  • Filter sheet: open filter → tap dropdown option → value selects correctly (users, organizations, fraud, email, activity pages)
  • Form pages at mobile width: /contacts/create, /contact-groups/create, /profile → card fills viewport
  • Timezone select at /profile: selected label truncates with ellipsis, chevron visible
  • Desktop: all pages unchanged

@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Apr 17, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@gaghan430 gaghan430 requested a review from a team April 17, 2026 05:14
@gaghan430 gaghan430 merged commit 387ff9d into main Apr 17, 2026
9 of 10 checks passed
@gaghan430 gaghan430 deleted the fix/mobile-filter-form-timezone branch April 17, 2026 09:01
@gaghan430 gaghan430 self-assigned this Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants