Skip to content

feat(ui): simplify header controls and remove redundant panels - #6

Merged
KnowSky404 merged 1 commit into
mainfrom
feat/simplify-ui
Oct 26, 2026
Merged

KnowSky404 merged 1 commit into
mainfrom
feat/simplify-ui

Conversation

@KnowSky404

@KnowSky404 KnowSky404 commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Problem and resulting behavior

The application header repeated connection badges, a Gist ID, synchronization labels, and a setup button. Page headers and side panels repeated the same status and counts.

Replace the header controls with three accessible icons for Workspace status/repair, theme, and GitHub. Remove redundant descriptions, summary badges, overview status panels, the Gist information sidebar and duplicate refresh control, and the export statistics card. Keep actionable warnings, recovery controls, publication links, and list counts. Overview counts link to their management pages; the header stays on one row on narrow screens.

Task and verification evidence

  • Worktree / branch: /root/Clouds/SubMan-ui-simplify / feat/simplify-ui.
  • Base SHA: 99ea56475914aa8ee5b3e06e91a3305059d79458.
  • Tested head SHA: 2c7f77901d030b47e95d264b0381fcc76678ea97.
  • Bun 1.3.14: 679 unit tests and 48 Worker-backed browser tests passed during implementation. Final verification after the last accessibility annotation: 62 route tests, type/Svelte and generated Worker type checks, lint, documentation checks, build, and four additional Playwright checks passed.
  • Browser verification: all six routes at 1440, 390, and 320 pixels; theme selection/persistence; status/settings navigation; overview links; no horizontal overflow, framework overlays, or browser errors. CI will rerun the complete suite on the submitted head.
  • Screenshots were inspected locally: /tmp/subman-ui-qa/after-overview-1440.png, after-overview-390.png, and after-dark.png. These local artifacts are not hosted attachments.
  • CI results for this head: verify passed, completed 2026-10-26 08:41:51 UTC. This includes unit tests, bilingual docs, sing-box validation, Svelte/types, lint, build, Cloudflare integration, deployment bundle validation, and browser tests.
  • Limitations: Chromium and mocked GitHub responses; no real Gist mutations or manual production deployment performed.

Review gate (update before merge)

  • Codex Code Review completed for the current head; author and SHA verified.
  • Security Review is still running. The user explicitly instructed direct merge after being informed of this pending review; this supersedes the repository instruction to wait for its completion for this PR. No GitHub ruleset or branch-protection gate is being bypassed.
  • Current-head review SHA, Code Review completion time, evidence links, and the pending Security Review exception are recorded below.
  • No Code Review findings or inline review findings were reported; no findings required resolution.
  • Required CI checks passed for the current head; new commits reviewed again.

Review evidence: Codex Review Summary, authored by chatgpt-codex-connector[bot]. Code Review completed at 2026-10-26 08:37:28.681729 UTC for 2c7f77901d030b47e95d264b0381fcc76678ea97, with no findings and no unresolved findings. Security Review remains running for the same head. After receiving this status, the user explicitly requested "直接合并吧" (merge directly), authorizing this PR to proceed without waiting for Security Review. Its pending state is not treated as a passing review. Submitted reviews and inline comments were inspected; none were present. Repository inspection found no main branch protection and no active rulesets; repository instructions still require completed current-head Codex review. The repository's preceding PR also ran Security Review, so both review types will be verified here.

Merge and completion (update after authorized merge)

  • Authorization: the user explicitly requested pushing and merging this UI change, then explicitly requested direct merge after being informed that Security Review was still running.
  • Strategy: rebase merge pinned to verified head 2c7f77901d030b47e95d264b0381fcc76678ea97; CI and Code Review passed; pending Security Review is covered by the explicit user override above.
  • PR: feat(ui): simplify header controls and remove redundant panels #6, merged at 2026-10-26 09:04:20 UTC.
  • Original PR head: 2c7f77901d030b47e95d264b0381fcc76678ea97; rebase merge SHA: e015b6f650abfe8c3aa89cbf9c741495f783e465.
  • Verified origin/main contains the merge SHA. git range-diff reports the original and rebased commit patches equal, and a full tree diff is empty (both trees 2944f8145d761870fea14a18b08dfb34f1b20b14). No conflict resolution or integration content changes required repeated checks.
  • The original clean /root/Clouds/SubMan checkout was fast-forwarded to e015b6f650abfe8c3aa89cbf9c741495f783e465; it remains clean on main.
  • Verified unchanged task head and no staged/unstaged/untracked changes or private configuration. Removed the task worktree and local branch. Deleted the remote branch using an explicit lease pinned to the original PR head. Validation logs/screenshots remain outside the repository at /tmp/subman-ui-qa.
  • No manual production deployment was invoked. Security Review was pending at merge under the user's explicit override recorded above.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T08:37:28.681729Z 2c7f779 PR opened
🔒 Security Review 🔄 Running since 2026-10-26T08:36:11.529012Z 2c7f779 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@KnowSky404
KnowSky404 merged commit e015b6f into main Oct 26, 2026
1 check passed
@KnowSky404
KnowSky404 deleted the feat/simplify-ui branch October 26, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant