Skip to content

fix(settings): keep controls reachable on small displays - #45

Merged
laceyp99 merged 3 commits into
mainfrom
t3code/blueprint-issue-31
Sep 7, 2026
Merged

laceyp99 merged 3 commits into
mainfrom
t3code/blueprint-issue-31

Conversation

@laceyp99

@laceyp99 laceyp99 commented Sep 7, 2026

Copy link
Copy Markdown
Owner

Summary

Fixes #31 by making every Settings tab vertically scrollable and fitting the window to the opening monitor's usable work area on constrained displays. The existing 760x700 default is retained where it fits. Windows tab scrolling now moves 60 pixels per wheel notch while slider value scrolling remains unchanged. Scrollable surfaces use matching light/dark colors.

Validation

  • venv\Scripts\python.exe -m ruff format --check .
  • venv\Scripts\python.exe -m ruff check .
  • venv\Scripts\python.exe -m pytest (167 passed)
  • Real-widget checks: default sizing, work-area sizing, 60-pixel wheel travel, slider isolation
  • Human verification: current display and 1366x768 at 100% and 150%; all tab controls reachable, no clipping, faster scrolling, reset isolation, Cancel/Save persistence, light/dark appearance

Notes

Ollama connection testing and secondary-monitor placement were not exercised. docs/plan.md and review.md remain local untracked artifacts.

@laceyp99
laceyp99 marked this pull request as ready for review September 7, 2026 03:27

@laceyp99 laceyp99 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Looks good to me. I was part of the interactive UI testing making sure in different display setups and everything mainly looked good. I'm glad I was able to catch the slow vertical scrolling and dark/light mode visibility support.

@laceyp99

laceyp99 commented Sep 7, 2026

Copy link
Copy Markdown
Owner Author

Did a quick second monitor test and nothing was caught as far as issues/bugs. Ollama is left untested as it isn't relevant to this patch.

@laceyp99
laceyp99 merged commit e1808ff into main Sep 7, 2026
6 checks passed
@laceyp99
laceyp99 deleted the t3code/blueprint-issue-31 branch September 7, 2026 03:33
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.

Settings window can clip content on small screens / high DPI (fixed geometry, no scrolling)

1 participant