Skip to content

feat(dives): twin sets sit below every single-cylinder group - #282

Merged
aleskiontherun merged 2 commits into
mainfrom
claude/twinsets-volume-autocomplete-1ae4de
Sep 23, 2026
Merged

aleskiontherun merged 2 commits into
mainfrom
claude/twinsets-volume-autocomplete-1ae4de

Conversation

@aleskiontherun

Copy link
Copy Markdown
Member

The cylinder volume dropdown groups its presets and leads with the ones in the reader's own unit system, which put Twin sets second for a metric diver and last for an imperial one. A twin set belongs to neither family — it is asked for by its total whichever way a diver reads — and it is the rarer pick, so the group now carries no system and trails both families in both systems. The reader's own singles stay nearest the box.

Verified by the two render tests asserting the full preset order in each unit system and the arrow-key walk over the first heading, plus the whole suite (7154 tests), lint, type-check, format and a production build.

🤖 Generated with Claude Code

aleskiontherun and others added 2 commits September 23, 2026 08:49
A twin set belongs to neither unit family - it is asked for by its total
whichever way a diver reads - and it is the rarer pick, so the group
carries no `system` and trails both families instead of riding in front of
the imperial presets for a metric reader.

Verified by the two render tests asserting the full preset order in each
unit system, plus the arrow-key walk over the first heading.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
The site comment carries why twin sets trail both families; the entry
states the order the dropdown renders in and nothing more.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@github-actions github-actions Bot added the feat New user-visible behaviour label Sep 23, 2026
@aleskiontherun
aleskiontherun merged commit 5fdbca9 into main Sep 23, 2026
12 checks passed
@aleskiontherun
aleskiontherun deleted the claude/twinsets-volume-autocomplete-1ae4de branch September 23, 2026 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New user-visible behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant