Skip to content

Fix connection modal height - #538

Merged
debba merged 3 commits into
TabularisDB:mainfrom
DhruvShah-Dev:agent/fix-connection-modal-height
Jul 28, 2026
Merged

Fix connection modal height#538
debba merged 3 commits into
TabularisDB:mainfrom
DhruvShah-Dev:agent/fix-connection-modal-height

Conversation

@DhruvShah-Dev

Copy link
Copy Markdown
Contributor

Summary

Validation

  • pnpm.cmd test tests/components/modals/NewConnectionModal.test.tsx --run
  • pnpm.cmd typecheck
  • pnpm.cmd lint
  • pnpm.cmd build
  • git diff --check
  • GitNexus impact and detect-changes: low risk, expected modal/test files only

@kilo-code-bot

kilo-code-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • src/components/modals/NewConnectionModal.tsx - 0 issues
  • tests/components/modals/NewConnectionModal.test.tsx - 0 issues

Reviewed by step-3.7-flash · Input: 76.5K · Output: 12.7K · Cached: 620.4K

@debba

debba commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Thanks a lot for this @DhruvShah-Dev! Heads up that a full restyling of the interface is coming soon, so some of this area will change, but this fix is a small and useful foundation to build on in the meantime.

If you feel like hanging out with the community, you're welcome to join our Discord: https://discord.com/invite/K2hmhfHRSt

@debba
debba merged commit 72f7364 into TabularisDB:main Jul 28, 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.

[Bug]:Dialog box height changes based on amount of content in the dialog box. Might be a better user experience if we fix this dialog

2 participants