Skip to content

fix(gui): chrome polish — tabs, Codex Auth load, steppers - #607

Closed
Wibias wants to merge 9 commits into
lidge-jun:devfrom
Wibias:fix/gui-chrome-forms
Closed

fix(gui): chrome polish — tabs, Codex Auth load, steppers#607
Wibias wants to merge 9 commits into
lidge-jun:devfrom
Wibias:fix/gui-chrome-forms

Conversation

@Wibias

@Wibias Wibias commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Dashboard + Logs/Debug: page tab strip wraps and no longer shows a pointless horizontal scrollbar
  • Startup: spacing between tray actions and the Windows tray error notice
  • Codex Auth: skeleton instead of top “Loading accounts” banner; quota refresh feedback stays in the page head (no Notice layout shift)
  • Rotation strategy spacing aligned with other setting cards; custom up/down steppers for switch threshold (+ sticky limit)
  • Providers note field ~30% taller; Grok collapse/expand all; API Keys active-keys skeleton while loading

Test plan

  • Dashboard / Logs: tab row has no horizontal scrollbar; wraps if needed
  • Startup: tray error notice has clear gap under buttons
  • Codex Auth: cold load shows skeleton cards (no top banner shift); Refresh quotas feedback in header
  • Switch threshold: chevron steppers work; wider gap from the number
  • Providers settings: note textarea taller
  • Grok: Collapse all / Expand all
  • API Keys: Active keys panel keeps height while loading

Part of the GUI UI/UX + loading split (UI-1). See docs/superpowers/plans/2026-07-28-gui-ux-perf-split.md on the local combine branch.

@Wibias
fix(gui): chrome polish — tabs, Codex Auth load, steppers
0a63029 
Remove pointless page-tab scrollbars, reserve Codex Auth/API Keys
load space without layout shift, tighten pool strategy spacing, and
add custom number steppers plus Grok collapse/expand controls.
@coderabbitai

coderabbitai Bot commented Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: b738f047-309c-4b5f-bb7a-440fb7264b7f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the bug Something isn't working label Jul 28, 2026
This was referenced Jul 28, 2026
Closed
Closed
Closed
Closed
Wibias added 8 commits July 28, 2026 21:33
@Wibias
fix(gui): stabilize Startup safety page chrome
acb4a42 
Add a Dashboard back link, commit runtime notice and tray actions with the health payload to avoid layout shift, and stack the Codex runtime warning vertically so the copy command no longer wraps beside the message.
@Wibias
fix(gui): underline-free Startup back link and service Repair action
b0a5868 
Remove link underline from a.btn, and show a Repair button when the background service or launcher shim is installed but not viable so stale protection can be fixed in-page.
@Wibias
fix(gui): paint Startup health before settings and tray
9ca00b3 
Show hero/details as soon as /api/startup-health returns, overlap settings
fetch with health, and reserve notice/tray slots so secondary fill-in does
not wait on the slow settings round-trip or shift layout.
@Wibias
fix(gui): seed Startup page from session cache for instant warm paint
4eed39b 
Reuse last health/notice/tray snapshot so revisiting #startup skips the
EmptyState spinner; refresh still revalidates in the background.
@Wibias
fix(service): repair installed background service without UAC
72f1adf 
GUI Repair was a label over full install (schtasks /create), which triggered
elevation. Add ocx service repair / API repair mode that rewrites assets and
restarts an already-registered service, matching the WinSW no-reprompt path.
@Wibias
fix(windows): decode schtasks /xml as UTF-16 so elevated install veri…
6613d55 
…fies

schtasks emits UTF-16LE for /query /xml; reading it as UTF-8 made every
post-elevate health check fail and rolled back a successful create.
@Wibias
fix(windows): accept elevated Task Scheduler XML rewrites on verify
d747ddd 
Elevated schtasks /create often rewrites RunLevel to HighestAvailable and
may change path casing or quote escaping; the post-install health check
treated that as unhealthy and rolled back. Also fall back to on-disk XML
when /query /xml returns empty.
@Wibias
fix(gui): use PowerShell-safe separator in Codex runtime fix command
9415bdd 
Windows PowerShell 5 rejects bash &&; copy ocx doctor; ocx sync on win32.
@Wibias

Wibias commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by a single combined GUI UX/perf PR from the local local/gui-ux-perf-bundle worktree (chrome + storage + dense workspaces + hot-path + page-data + later polish).

Closing this split PR to avoid duplicate review; the replacement PR will land against dev shortly.

@Wibias Wibias closed this Jul 30, 2026
@Wibias
Wibias deleted the fix/gui-chrome-forms branch July 30, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lidge-jun lidge-jun Awaiting requested review from lidge-jun lidge-jun will be requested when the pull request is marked ready for review lidge-jun is a code owner
@Ingwannu Ingwannu Awaiting requested review from Ingwannu Ingwannu will be requested when the pull request is marked ready for review Ingwannu is a code owner

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Wibias