Skip to content

[Bug]: Terminal drawer toggle has no spacing from "Commit, push & PR" and its hover/tooltip overlaps it #13419

Closed

Description

@tinyfroggy

Before submitting

  • I searched existing issues and did not find a duplicate.
  • I included enough detail to reproduce or investigate the problem.

Area

apps/desktop

Steps to reproduce

  1. Open the T3 Code desktop app (Linux) and open a project/thread.
  2. Look at the top toolbar: Add action, Open, Commit, push & PR, then the terminal drawer toggle (tooltip "Toggle terminal drawer (Ctrl+J)").
  3. Hover the terminal drawer toggle.

Expected behavior

The terminal drawer toggle should have the same spacing/margin as the other toolbar controls, and its hover state/tooltip should stay inside its own bounds and not overlap the Commit, push & PR button.

Actual behavior

There is no margin between the terminal drawer toggle button and the Commit, push & PR button. When hovering, the hover state/tooltip is drawn on top of the Commit, push & PR button.

Impact

Cosmetic issue

Version or commit

0.0.43-nightly.20260924.2200 (x86_64 AppImage, build 473ee9c)

Environment

CachyOS (Arch-based), Linux kernel 7.2.6-1-cachyos, x86_64; T3 Code desktop app (Electron). Report filed via opencode using model opencode-go/deepseek-v4.1-flash.

Screenshots, recordings, or supporting files

Image

Activity

  1. added
    bugSomething is broken or behaving incorrectly.
    needs-triageIssue needs maintainer review and initial categorization.
    on Sep 24, 2026
  2. juliusmarminge commented on Sep 24, 2026

    @juliusmarminge
    Member

    Confirmed. The terminal toggle is not in the header action row, so it does not get the same gap as Add action / Open / Commit, push & PR, and its tooltip paints across that button.

    Header actions only reserve a 4px clearance for two toggles (ChatHeader, pr-[calc(--spacing(18)+1px)] sm:pr-[calc(--spacing(14)+1px)]). The toggles are a separate fixed cluster (ChatView, data-workspace-titlebar-controls). Above 980px that cluster still mounts the right-panel maximize control while the panel is closed: opacity-0 and inert, but it remains a flex item (one sm toggle plus gap-1, about 32px). That width is not in the reserved padding, so the terminal toggle sits against Commit, push & PR. The tooltip (PanelLayoutControls, side="bottom") is wider than the toggle and draws over the button. In the screenshot its box covers the right half of Commit, push & PR and is centered about one toggle-slot left of the glyph.

    Same header on web and desktop. Reproduced from the screenshot on 0.0.43-nightly.20260924.2200 (b2b43bef); the reported build id 473ee9c does not resolve.

    Fix should drop the closed maximize control out of the flex layout (not only hide it) and reserve the same gap as the other header actions (gap-2 / gap-3, not 4px). After that, check the tooltip still anchors to the terminal toggle rather than the hidden slot.

  3. added
    acceptedfeature request accepted
    via-triageFiled through npx t3 triage
    and removed
    needs-triageIssue needs maintainer review and initial categorization.
    on Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    acceptedfeature request acceptedbugSomething is broken or behaving incorrectly.via-triageFiled through npx t3 triage

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions