Skip to content

Polish stack view - #9000

Open
Alex Ross (alexr00) wants to merge 2 commits into
alexr00/cautious-marsupialfrom
alexr00/prime-boar
Open

Alex Ross (alexr00) wants to merge 2 commits into
alexr00/cautious-marsupialfrom
alexr00/prime-boar

Conversation

@alexr00

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 UI changes are consistent and adequately covered by tests.

Review effort: Balanced
Findings: None

What changed in this PR

Polishes stack status visuals and adjusts branch deletion layout for stacked pull requests.

Changes:

  • Adds position-aware stack readiness icons and colors.
  • Moves stacked branch deletion into a dedicated block.
  • Expands UI tests for icons and button placement.
File Description
webviews/​editorWebview/​test/​overview.test.tsx Tests stack icons and branch deletion placement.
webviews/​editorWebview/​index.css Styles closed states and stacked deletion layout.
webviews/​components/​pullRequestStack.tsx Selects refined readiness icons by state and position.
webviews/​components/​merge.tsx Marks stacked branch deletion containers.

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

@alexr00
Alex Ross (alexr00) removed this pull request from stack #8993 October 1, 2026 14:33
@alexr00
Alex Ross (alexr00) added this pull request to stack #9001 October 1, 2026 14:35
@alexr00
Alex Ross (alexr00) marked this pull request as ready for review October 1, 2026 14:35
Copilot AI balanced review requested due to automatic review settings October 1, 2026 14:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 visual changes are consistent and adequately covered by tests.

Review effort: Balanced
Findings: None

Copilot AI balanced review requested due to automatic review settings October 1, 2026 14:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

The closed-state icon becomes invisible in high-contrast themes.

Review effort: Balanced
Findings: None

Previously missed (1)

In code that hasn't changed since last review

Medium severity Add high-contrast foreground override for blocked icon

webviews/​editorWebview/​index.css:795

In high-contrast themes, pullRequests.closed defaults to editor.background (package.json:3904-3909). Because non-current readiness badges also use editor.background, this new blocked icon becomes invisible. Please give blocked the same high-contrast foreground override already applied to merged and draft states.

roblourens
roblourens previously approved these changes Oct 1, 2026
@alexr00
Alex Ross (alexr00) removed this pull request from stack #9001 October 2, 2026 18:51
Copilot AI balanced review requested due to automatic review settings October 3, 2026 10:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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

The closed-state icon can become invisible in high-contrast themes.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)


.stack-entry-readiness.blocked {
color: var(--vscode-errorForeground);
color: var(--vscode-pullRequests-closed);
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.

3 participants