Skip to content

Keep Upgrades Source visible and columns stable - #35

Merged
harder merged 2 commits into
mainfrom
codex/upgrades-source-width
Oct 6, 2026
Merged

harder merged 2 commits into
mainfrom
codex/upgrades-source-width

Conversation

@harder

@harder harder commented Oct 6, 2026 •

Copy link
Copy Markdown
Owner

The v0.2.1 Upgrades screen clips the Source column to winge in a 150-column terminal. Reserve enough rendering space for Source and refresh TableView content size when column styles change, so the first render and subsequent selections use the same widths.

This branch also refreshes the website media from the fixed TUI: four theme screenshots at 140×34 cells and two short GIFs for Search and Upgrades. All captures use --mock and show sample data from this development branch. The README and site now display the new media. Public and contributor docs were updated to distinguish current ZIP/EXE downloads from the unreleased MSIX path and to remove stale version/setup text.

Validation:

  • 316 .NET tests passed on Windows ARM64.
  • python site/build.py passed, including local files and anchors.
  • Local Markdown targets and git diff --check passed.
  • Captures were visually inspected; the Upgrades Source column shows winget in full.

The site will deploy through GitHub Pages when this PR is merged to main.

Copilot AI balanced review requested due to automatic review settings October 6, 2026 02:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The focused layout fix has regression coverage and reported manual TUI validation.

Review effort: Balanced
Findings: None

What changed in this PR

Keeps the Upgrades Source column visible and stable across initial rendering and selection changes.

Changes:

  • Reserves explicit Source and rendering-gutter space.
  • Refreshes cached table dimensions after width changes.
  • Adds a regression test for stable content width.
File Description
src/​App.cs Corrects column budgeting and refreshes content size.
tests/​AppBehaviorTests.cs Verifies Upgrades width stability at 150 columns.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@harder
harder requested a balanced review from Copilot October 6, 2026 15:48
@harder
harder merged commit 8bedb80 into main Oct 6, 2026
13 checks passed
@harder
harder deleted the codex/upgrades-source-width branch October 6, 2026 15:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Both new autoplaying GIFs loop for longer than five seconds without user controls.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)

Comment thread site/index.html
</div>
</div>
<div class="demo-grid" aria-label="Scout demonstrations">
<figure class="demo-card"><img src="media/scout-search-demo.gif" width="1405" height="805" loading="lazy" alt="Scout opens Search, finds Git, and moves between results while package details stay visible"><figcaption><strong>Find a package</strong><span>Search and inspect without losing the results list.</span></figcaption></figure>
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