Skip to content

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

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

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.

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

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.

Wibias added 8 commits July 28, 2026 21:33
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.
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.
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.
Reuse last health/notice/tray snapshot so revisiting #startup skips the
EmptyState spinner; refresh still revalidates in the background.
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.
…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.
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.
Windows PowerShell 5 rejects bash &&; copy ocx doctor; ocx sync on win32.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant