DATASHARE-158 Government Usage Mobile Responsive Behavior#89
Merged
RuoxiZhang08 merged 1 commit into1.2.0from Mar 9, 2026
Merged
DATASHARE-158 Government Usage Mobile Responsive Behavior#89RuoxiZhang08 merged 1 commit into1.2.0from
RuoxiZhang08 merged 1 commit into1.2.0from
Conversation
There was a problem hiding this comment.
Pull request overview
Improves the Government Usage “OverlayWindow” modal layout to behave better on smaller viewports by constraining width/height and enabling internal scrolling while preserving the existing dialog structure.
Changes:
- Add outer padding and convert the modal container to
w-fullwith amax-wconstraint for better mobile responsiveness. - Constrain modal height with
max-hand prevent content overflow by moving scrolling into the body area (flex-1 min-h-0 overflow-auto). - Update the Vitest snapshot to reflect the new class names.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
src/components/OverlayWindow/index.tsx |
Adjusts modal sizing/overflow styles for improved mobile responsiveness. |
src/components/OverlayWindow/__snapshots__/index.test.tsx.snap |
Updates snapshot output to match the updated modal markup/classes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
RuoxiZhang08
approved these changes
Mar 9, 2026
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
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.
Overview
This PR fixes the missing mobile responsive behavior for the government usage banner that was lost during the MUI->tailwind migration.
Change Details (Specifics)
Related Ticket(s)
DATASHARE-158