Skip to content

Clarify web conversion flow and refresh comparison previews - #26

Merged
Stanback merged 7 commits into
mainfrom
design/converter-layout-exploration
Oct 19, 2026
Merged

Stanback merged 7 commits into
mainfrom
design/converter-layout-exploration

Conversation

@Stanback

@Stanback Stanback commented Oct 19, 2026 •

Copy link
Copy Markdown
Owner

The web converter previously opened on an unexplained sample time and presented inputs/results as separate tall cards. This layout study starts with a blank time and an explicit “Try an example” action, groups the desktop comparison, and keeps editable inputs before results on phones.

The three course results have clearer selected/original states, with restrained hover borders that preserve the selected fill. The numerical panel stays static. Calculation factors and the source explanation sit in a keyboard-accessible disclosure, closed initially, with a route to full methodology. Phone rows have tighter vertical spacing and deliberate save/copy/download action rows; the reviewed 390×844 comparison fits all actions above navigation, while shorter and larger-text views scroll normally. A compact next-standard shortcut uses the existing youth table and the unadjusted entered swim; altitude/other systems route to the full benchmark view. NCAA stays the default and Colorado remains a quiet optional setting. Shared URLs and saved/recent comparisons still restore their explicit swims. Reset clears the web time; remembered swimmer settings retain their existing contract. Loading the demonstration alone does not add conversion history.

The README now leads with an actual-build converter crop showing inputs, all three courses and the next standard. The detailed review contains matching light/dark, phone arrival/input/result, Colorado and narrow long-time captures, with source hashes. Existing paired native home captures were reviewed and retained: native layout and default sample behavior are unchanged in this web PR. No widget/MCP presentation or calculation engine changed.

Positioning now leads with a private offline swim companion: compare courses, understand published benchmarks and keep a small local scratchpad. README/product notes distinguish web saved comparisons from native development builds’ actual practice/meet records. General logger expansion, cloud sync, coach dashboards and hosted aggregation remain outside the roadmap. Native colors/icons are shared; this web layout has not automatically propagated to SwiftUI/Compose.

Validation:

  • Svelte/TypeScript: zero errors/warnings.
  • 138 unit tests pass.
  • 43 desktop/mobile browser cases pass, one desktop-inapplicable case skipped. Coverage includes neutral blank arrival, explicit example, reset, no demonstration history, stable input focus, keyboard disclosure behavior, phone action visibility, complete 320px category/age labels and enlarged-root-text reflow.
  • 20 production/offline cases pass, including hydration, service-worker behavior, shared swims, widget isolation and setup pages.
  • Light/dark and narrow captures inspected; formatting, whitespace, relative links and capture-source hashes checked.

The owner approved the web design on October 19, 2026. All hosted checks passed on the reviewed implementation, including the polish pass. The branch is now synchronized with main for final merge checks. Paired native input-first/blank behavior, new benchmark datasets and experimental prediction graphics are separate work, not shipped by this PR.

Actual-build review · Desktop comparison · Benchmark shortlist

@Stanback Stanback changed the title Explore calmer converter layout and refresh README screenshots Clarify web conversion flow and refresh comparison previews Oct 19, 2026
@Stanback
Stanback marked this pull request as ready for review October 19, 2026 21:08
@Stanback
Stanback enabled auto-merge (squash) October 19, 2026 21:09
@Stanback
Stanback merged commit ba950cc into main Oct 19, 2026
11 checks passed
@Stanback
Stanback deleted the design/converter-layout-exploration branch October 19, 2026 21:13
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