Skip to content

Plan: UI discoverability pass (Experiments panel)#2

Open
shaal wants to merge 1 commit intomainfrom
ui-discoverability-pass
Open

Plan: UI discoverability pass (Experiments panel)#2
shaal wants to merge 1 commit intomainfrom
ui-discoverability-pass

Conversation

@shaal
Copy link
Copy Markdown
Owner

@shaal shaal commented Apr 24, 2026

Summary

This PR contains a plan document only — no feature code. It
proposes a UI discoverability pass for the features landed in PR #1
(RuLake-inspired roadmap).

The core issue: three features (`?snapshots=1` save/share,
`?crosstab=1` cross-tab sync, and the library-only 1-bit
quantization) ship with zero UI — only discoverable by reading
docs or chapter text. Two other features (federation, consistency
modes) already use the right pattern: URL flag presets the initial
state of an always-visible UI control.

This plan proposes a single `🧪 Experiments` disclosure panel that:

  1. Consolidates all feature toggles into one collapsible section at the bottom of the training panel.
  2. Un-gates the currently-hidden snapshot/share and cross-tab rows — render always, default OFF, flag continues to work as a preset.
  3. Adds a disabled row for 1-bit quantization with a tooltip explaining it's library-only.
  4. Adds a `confirm()` guardrail on destructive Import actions.
  5. Adds a `[Learn]` link per row that calls `window.ELI15.openChapter(id)`.

Scope is small: ~250 lines, one PR, primarily in `AI-Car-Racer/uiPanels.js`. No feature module is touched.

See `docs/plan/ui-discoverability-pass.md` for:

  • 10-task checklist status tracker
  • Default-state table per feature
  • Exit criteria + harness requirements
  • Design trade-offs (native `
    Details` over custom widgets, one consolidated panel over scattered toggles, no default-on flips)

Why plan-only first

Per the working discipline on this project, non-trivial UX
changes get a reviewable plan document before code. This PR is
the reviewable plan; the implementation PR will follow once the
plan is approved.

What's NOT in this plan

  • No new URL flags.
  • No default-on flips (Federation, Cross-tab, Snapshots, Quantization all stay OFF by default).
  • No redesign of the existing training panel.
  • No integration of quantization into the archive hot path.
  • No real community archive URLs published.

Test plan

  • Review `docs/plan/ui-discoverability-pass.md` end-to-end.
  • Confirm the default-state table matches the roadmap's conservatism posture (plain URL = plain behaviour).
  • Confirm the exit criteria are testable via agent-browser.
  • Approve or request specific changes to the design calls (single-panel vs. scattered, default-off vs. default-on, native vs. custom widgets).

After approval, an implementation PR lands the actual ~250 lines + the `tests/experiments-panel-smoke.html` harness.

Follow-up plan for the RuLake-inspired roadmap addressing the
single largest UX weakness: three features (warm-restart/share,
cross-tab, and the absent quantization UI) ship with zero
discoverability because their UI is gated on URL flags.

Proposes a single 🧪 Experiments disclosure panel consolidating
all feature toggles, with URL flags demoted from UI-gate to
initial-state-preset (the pattern consistency-modes and federation
already use correctly). No default-on flips; plain URL still yields
plain behaviour.

Scope is deliberately small: ~250 lines, primarily in uiPanels.js,
with a new agent-browser smoke harness. No feature module touched.

See the plan for the 10-task checklist, default-state table, and
design trade-offs.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 24, 2026

Warning

Rate limit exceeded

@shaal has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 57 minutes and 44 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 57 minutes and 44 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9f737d0d-fdbb-4b15-896d-2937491d9111

📥 Commits

Reviewing files that changed from the base of the PR and between ee2dd34 and d778440.

📒 Files selected for processing (1)
  • docs/plan/ui-discoverability-pass.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ui-discoverability-pass

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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