Skip to content

fix(web): contain overflowing modal inputs in Add Repository dialog (#843)#844

Merged
RaghavChamadiya merged 4 commits into
repowise-dev:mainfrom
jyotirmya17:fix/modal-alignment
Jul 15, 2026
Merged

fix(web): contain overflowing modal inputs in Add Repository dialog (#843)#844
RaghavChamadiya merged 4 commits into
repowise-dev:mainfrom
jyotirmya17:fix/modal-alignment

Conversation

@jyotirmya17

Copy link
Copy Markdown
Contributor

Fixes the layout overflow reported in #843, where input fields inside the "Add Repository" modal extended past the modal's container bounds.

Root Cause

The modal's inputs were missing two properties needed for correct sizing inside a flex/grid layout:

  • No box-sizing: border-box equivalent, so horizontal padding was added on top of the declared 100% width instead of being included in it.
  • No min-width: 0, so inputs couldn't shrink below their browser-default intrinsic width once the flex/grid container ran out of space.

Changes

  • Added box-border to Input and SelectTrigger — ensures padding is included in the 100% width calculation instead of expanding past it.
  • Added min-w-0 to the inputs — allows them to shrink correctly inside flex/grid layouts instead of overflowing at their intrinsic minimum width.
  • Added overflow-hidden to the modal container as a failsafe — the two fixes above resolve the root cause, but this guards against similar overflow if new fields are added to the modal in the future without the same width handling.

Screenshots

Before:
image

After:
image

Fixes #843

@repowise-bot

repowise-bot Bot commented Jul 15, 2026

Copy link
Copy Markdown

✅ Health: 7.6 (unchanged)

📋 At a glance
1 hotspot touched. Scoped to packages.

🔎 More signals (1)

🔥 Hotspot touched (1)

  • .../onboarding/add-repo-wizard.tsx — 1 commits/90d, 3 dependents · primary owner: Raghav Chamadiya (100%)

📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-15 07:15 UTC
Silence on a single PR with [skip repowise] in the title · Per-repo toggle on repowise.dev/settings?tab=bot

@RaghavChamadiya RaghavChamadiya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@RaghavChamadiya RaghavChamadiya merged commit dfa4cd5 into repowise-dev:main Jul 15, 2026
7 checks 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.

[Bug] Add Repository modal inputs overflow container bounds

2 participants