Skip to content

fix(ui): give dialogs more vertical room on larger screens - #108

Merged
Go2Engle merged 1 commit into
mainfrom
feature/taller-dialogs
Oct 11, 2026
Merged

Go2Engle merged 1 commit into
mainfrom
feature/taller-dialogs

Conversation

@Go2Engle

Copy link
Copy Markdown
Owner

Summary

Projects, Settings, and other dialogs can use more vertical room on larger screens, so longer lists and pages show more content before scrolling. Short dialogs stay compact, and close/save controls in Projects and Settings remain visible while their content scrolls.

Type of change

  • Feature
  • Bug fix
  • Refactor
  • Documentation
  • Infrastructure/CI

Technical details

  • Use one viewport-based dialog height limit with 64 pixels of vertical clearance, reduced to 28 pixels on narrow windows.
  • Remove the project list's fixed 360-pixel height cap and allow Settings pages to grow beyond 900 pixels when their content needs the room.
  • Keep dialog headings and footers from shrinking, and let card search content shrink and scroll in shorter windows.
  • Allow the artwork choice panel to grow with the available dialog height.
  • Update the feature and installation guides and refresh the Settings screenshot.

Testing performed

  • Application tests: 204 passed after updating the branch from main.
  • Website tests: 17 passed; offline website build verified 12 pages and 19 releases.
  • Browser checks at 2560 × 1440, 1280 × 720, 390 × 700, and 1024 × 480 covered long project lists, long Settings release notes, registered printing, card search, and compact confirmation dialogs. Close/save controls remained accessible.
  • Prettier and Git whitespace checks passed.
npm test
npm run build
npm ci --prefix site
npm test --prefix site
npm run build --prefix site -- --offline
npm run check:docs --prefix site
npx prettier --check src/style.css
git diff --check

Breaking changes

None.

Checklist

  • The PR title follows Conventional Commits.
  • I ran npm test and npm run build.
  • I added or updated tests for behavior changes. This CSS change was verified through browser layout checks.
  • I updated documentation for user-facing changes.
  • I reviewed the README feature summary and workflow; they remain current.
  • I did not manually edit release versions or CHANGELOG.md.

@Go2Engle
Go2Engle merged commit c2e078b into main Oct 11, 2026
7 checks passed
@Go2Engle
Go2Engle deleted the feature/taller-dialogs branch October 11, 2026 01:01
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.

1 participant