Skip to content

#9 Fix tmux pane window size detection#13

Open
bsutton wants to merge 1 commit into
mainfrom
issue-9-tmux-window-size
Open

#9 Fix tmux pane window size detection#13
bsutton wants to merge 1 commit into
mainfrom
issue-9-tmux-window-size

Conversation

@bsutton
Copy link
Copy Markdown

@bsutton bsutton commented May 29, 2026

Summary

  • query Unix terminal dimensions with TIOCGWINSZ before falling back to Dart stdout terminal size
  • report Windows visible console window dimensions through GetConsoleScreenBufferInfo
  • add an Unreleased changelog entry for issue Window size from Tmux #9

Fixes #9.

Validation

  • dart analyze
  • dart test
  • live TTY smoke run: /home/bsutton/apps/flutter/bin/cache/dart-sdk/bin/dart run example/main.dart

Note: tmux is not installed in this environment, so I could not run an end-to-end tmux pane check locally.

@bsutton
Copy link
Copy Markdown
Author

bsutton commented May 29, 2026

Additional validation after installing tmux:

  • Started detached tmux sessions with forced pane sizes.
  • reported in a 41x13 pane.
  • reported in a 57x17 pane.
  • Removed the temporary local smoke-check script afterward; working tree is clean.
  • Analyzing dart_console...
    No issues found! still passes.

@bsutton
Copy link
Copy Markdown
Author

bsutton commented May 29, 2026

Correction to the previous validation note: the tmux checks passed after installing tmux.

  • Started detached tmux sessions with forced pane sizes.
  • Console.windowWidth/windowHeight reported 41 13 in a 41x13 pane.
  • Console.windowWidth/windowHeight reported 57 17 in a 57x17 pane.
  • Removed the temporary local smoke-check script afterward; working tree is clean.
  • dart analyze still passes.

@bsutton bsutton mentioned this pull request May 29, 2026
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.

Window size from Tmux

1 participant