Skip to content

Adjust prompt rail pipes#442

Open
aravhawk wants to merge 3 commits intoKilo-Org:devfrom
aravhawk:dev
Open

Adjust prompt rail pipes#442
aravhawk wants to merge 3 commits intoKilo-Org:devfrom
aravhawk:dev

Conversation

@aravhawk
Copy link

Context

Fixes a TUI rendering bug where the vertical bar next to the prompt/agent row is partially cut off in small terminals (for example 80x24 in Terminal.app).

Closes #438.

Implementation

Updated prompt border glyph rendering in /Users/aravhawk/kilo/packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx to keep the rail visually continuous in compact terminal sizes:

  • Main prompt container keeps a full-height left rail () through the seam.
  • The separator row below uses a half-height left glyph () to render the intended “4.5-cell” tail instead of a clipped or detached full segment.
  • Kept layout/spacing logic unchanged (no additional width/height conditionals), so this remains a minimal visual fix.

Screenshots

before after
before after

How to Test

  1. Open a small terminal window (for example 80x24).
  2. Run Kilo:
    • installed CLI: kilo
    • or from repo: cd /Users/aravhawk/kilo && bun run dev
  3. Inspect the left vertical rail beside the prompt box.
  4. Confirm it is no longer cut off and the bottom tail renders as a half segment instead of a detached/clipped full segment.
  5. Resize to a larger terminal and confirm the prompt rail still looks correct.

Get in Touch

Discord: @aravhawk

@github-actions
Copy link
Contributor

Hey! Your PR title Adjust prompt rail pipes doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, kilo).

See CONTRIBUTING.md for details.

customBorderChars={{
...EmptyBorder,
vertical: theme.backgroundElement.a !== 0 ? "╹" : " ",
// kilocode_change start - render only the seam half in the final row for a 4.5-cell rail
Copy link
Contributor

Choose a reason for hiding this comment

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

WARNING: Removed alpha-transparency conditional may cause visual regression

The old code was:

vertical: theme.backgroundElement.a !== 0 ? "╹" : " ",

This conditionally hid the glyph when the background element was fully transparent (a === 0). The new code always renders "╹" regardless of alpha.

Notably, the sibling <box> element just below (around line 1048-1057 in the base) still uses the same theme.backgroundElement.a !== 0 conditional to decide between horizontal: "▀" and horizontal: " ". This creates an inconsistency: when the theme has a transparent background element, the horizontal border will be hidden but the vertical rail glyph will still render.

Was this intentional? If so, please confirm. If not, consider preserving the conditional:

vertical: theme.backgroundElement.a !== 0 ? "╹" : " ",

@kiloconnect
Copy link
Contributor

kiloconnect bot commented Feb 18, 2026

Code Review Summary

Status: No New Issues Found | Recommendation: Merge (after addressing existing comment)

Overview

This PR makes two small cosmetic changes to the TUI prompt border characters in packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx:

  1. Line 811: Changes bottomLeft border char from "╹" (half-height bottom) to "┃" (full vertical bar) to keep the rail continuous to the seam.
  2. Line 1044: Simplifies the vertical border char from a conditional theme.backgroundElement.a !== 0 ? "╹" : " " to always use "╹". An existing review comment already flags the concern about removing the alpha-transparency conditional.

No security, logic, or runtime issues were identified beyond the existing inline comment.

Files Reviewed (1 file)
  • packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx - 0 new issues (1 existing comment)

@vercel
Copy link

vercel bot commented Feb 19, 2026

@aravhawk is attempting to deploy a commit to the Kilo Code Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vertical bar near input field is split in small terminal windows

1 participant

Comments