Delete standalone fleet-tab-strip crate#107
Merged
NagyVikt merged 1 commit intoMay 15, 2026
Merged
Conversation
This was referenced May 16, 2026
NagyVikt
added a commit
that referenced
this pull request
May 18, 2026
…ck redesign (#197) The legacy `fleet-tab-strip` Rust binary that used to draw an in-window header pane at the top of the overview window was retired in PR #107, but `full-bringup.sh` and `overview-header.sh` still hunted for the missing binary, hit a warn branch, and silently set `HEADER_ROWS=0` — so every fresh bringup came up with no top navigation header attached. Fix: - full-bringup.sh: default `CODEX_FLEET_OVERVIEW_HEADER_ROWS` to 0, drop the dead binary lookup, and upgrade the chrome-verify block from a bare `warn` to an auto-correct that re-asserts `status on` + `status-position top` (with `refresh-client -S`) if drift is detected. - overview-header.sh: collapse to a no-op + log line — the tmux status bar (configured top-docked by style-tabs.sh) is now the single canonical nav surface, visible on every window not just overview. Redesign (parallel 3-agent design pass, picked Design A "Glass-pill dock"): - Strip bg lifted from `#000000` to `#0A0A0C` so the `#1C1C1E` glass chips have a perceptible vertical lift (the "translucent dock" cue). - Inactive tabs: rounded `╭ ╮` caps in the chip-fill color → reads as a glass card sitting on the dock surface. - Active tab: half-block `▌ ▐` caps in iOS-blue → reads as a backlit dock chip lifted out of the strip surface. The mixed cap glyph family (rounded vs slab) gives strong focus-state contrast at peripheral vision without needing a second row. - Status-left (session badge) + status-right (live pill, clock with `◷` glyph) re-flowed onto the lifted bg. - Single-row preserved → `MouseDown1Status` tab clicks still fire (tmux 3.6 multi-row mouse-routing quirk avoided). Co-authored-by: NagyVikt <nagy.viktordp@gmail.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
rust/fleet-tab-stripcrate files:rust/fleet-tab-strip/Cargo.tomlrust/fleet-tab-strip/src/main.rsrust/Cargo.tomlunchanged; the workspacefleet-*glob now omits the crate because its manifest is gone.rust/Cargo.lock, which removed only thefleet-tab-strippackage entry.Verification
Baseline metadata before deletion included:
Metadata after deletion:
Workspace check:
Notes:
cargo check -p fleet-tab-striporcargo test -p fleet-tab-stripafter deletion because the package is intentionally removed by this lane.fleet_ui::tab_stripremoval remains Lane 7 scope.