Skip to content

Simplify Drawer positioning with Tailwind inset shorthand#47

Merged
mrlhnf merged 1 commit intofeature/SSD-760-fix-missing-contentfrom
copilot/sub-pr-46
Jan 15, 2026
Merged

Simplify Drawer positioning with Tailwind inset shorthand#47
mrlhnf merged 1 commit intofeature/SSD-760-fix-missing-contentfrom
copilot/sub-pr-46

Conversation

Copy link
Contributor

Copilot AI commented Jan 14, 2026

Replaced verbose after pseudo-element positioning in the Drawer component's mobile view with Tailwind's inset shorthand.

Changes:

  • Consolidated after:!bottom-0 after:!left-0 after:!right-0 after:!top-0 to after:!inset-0 in DrawerContent

The change maintains identical styling while improving code readability and reducing the class string length for the mobile drawer component.

// Before
className={cn(
  "fixed inset-x-0 bottom-0 z-50 flex max-h-[90dvh] min-w-[360px] flex-col overflow-auto rounded-t-xl border border-otl-gray-200 bg-bg-white after:!bottom-0 after:!left-0 after:!right-0 after:!top-0 after:!h-0",
  className
)}

// After
className={cn(
  "fixed inset-x-0 bottom-0 z-50 flex max-h-[90dvh] min-w-[360px] flex-col overflow-auto rounded-t-xl border border-otl-gray-200 bg-bg-white after:!inset-0 after:!h-0",
  className
)}

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix the mobile view of drawer hansard catalogue Simplify Drawer positioning with Tailwind inset shorthand Jan 14, 2026
Copilot AI requested a review from mrlhnf January 14, 2026 08:27
@harrisazmi harrisazmi marked this pull request as ready for review January 15, 2026 06:38
@mrlhnf mrlhnf merged commit 56d8a0b into feature/SSD-760-fix-missing-content Jan 15, 2026
1 check passed
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