Skip to content

refactor: css improvements#444

Open
ethanluc7 wants to merge 5 commits intomainfrom
refactor/css-improvements
Open

refactor: css improvements#444
ethanluc7 wants to merge 5 commits intomainfrom
refactor/css-improvements

Conversation

@ethanluc7
Copy link
Copy Markdown
Contributor

Summary

Improves UI styling and layout across various components such as the Table, Dropdown and EntityRelationshipDiagram components.

Description of Changes

  • Dropdown: Prevent dropdowns from stretching to fill parent width by adding align-self: flex-start, width: fit-content, and max-width: 100%
  • Modal: Increase max height from 80vh to 100vh to use full viewport
  • Diagram Modal: Increase diagram container height from 50vh to 65vh for better visibility
  • Table columns: Fix inconsistent first-column sizing by adding explicit width: 325px to both TableHeader and TableRow sticky columns
  • ERD layout: Swap sugiyama center nodes in mapping for improved diagram readability

Readiness Checklist

  • Self Review
    • I have performed a self review of code
    • I have run the application locally and manually tested the feature
    • I have checked all updates to correct typos and misspellings
  • Formatting
    • Code follows the project style guide
    • Autmated code formatters (ie. Prettier) have been run
  • Local Testing
    • Successfully built all packages locally
    • Successfully ran all test suites, all unit and integration tests pass
  • Updated Tests
    • Unit and integration tests have been added that describe the bug that was fixed or the features that were added
  • Documentation
    • All new environment variables added to .env.schema file and documented in the README
    • All changes to server HTTP endpoints have open-api documentation
    • All new functions exported from their module have TSDoc comment documentation

@ethanluc7 ethanluc7 requested a review from JamesTLopez April 13, 2026 13:29
Copy link
Copy Markdown

@JamesTLopez JamesTLopez left a comment

Choose a reason for hiding this comment

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

LGTM!

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