fix: keep slippage preset chips in one aligned group #1064

Merged

PlasticDigits merged 2 commits from feat/528-slippage-preset-align into main

2026-08-16 08:46:26 +00:00

PlasticDigits commented

2026-08-16 08:33:56 +00:00

(Migrated from gitlab.com)

Summary

Shared SlippageProtectionPresets puts the Slippage protection label above a role="group" 3-up grid so 0.5% / 1% / 5% stay one aligned row on /trade Market and Swap Settings.
Swap Custom stacks below that group (never between chips). Preset values, default 5%, clamp, and max_spread = percent/100 are unchanged.
Documents invariants S528-1–S528-10 in docs/frontend.md and the #528 agent playbook.

Closes #528

Test plan

VERIFY_ISSUE_528_SKIP_E2E=1 make verify-issue-528 (8/8)
Vitest scoped suite (65 tests) — presets, default 5%, Custom sanitize, submit max_spread
Playwright P1–P10 Chromium (e2e/slippage-preset-align-528.spec.ts, 5 workers) against LocalTerra
Manual: /trade Market at phone + squeezed desktop ticket — chips share one row under the label
Manual: Swap Settings Custom still works; 99 clamps/warns; 0 range error
Manual: keyboard Tab 0.5 → 1 → 5; focus ring; light + dark

## Summary - Shared `SlippageProtectionPresets` puts the **Slippage protection** label above a `role="group"` 3-up grid so **0.5% / 1% / 5%** stay one aligned row on `/trade` Market and Swap Settings. - Swap **Custom** stacks below that group (never between chips). Preset values, default **5%**, clamp, and `max_spread = percent/100` are unchanged. - Documents invariants **S528-1–S528-10** in `docs/frontend.md` and the #528 agent playbook. Closes #528 ## Test plan - [x] `VERIFY_ISSUE_528_SKIP_E2E=1 make verify-issue-528` (8/8) - [x] Vitest scoped suite (65 tests) — presets, default 5%, Custom sanitize, submit `max_spread` - [x] Playwright P1–P10 Chromium (`e2e/slippage-preset-align-528.spec.ts`, 5 workers) against LocalTerra - [ ] Manual: `/trade` Market at phone + squeezed desktop ticket — chips share one row under the label - [ ] Manual: Swap Settings Custom still works; `99` clamps/warns; `0` range error - [ ] Manual: keyboard Tab 0.5 → 1 → 5; focus ring; light + dark

PlasticDigits commented

2026-08-16 08:34:02 +00:00

(Migrated from gitlab.com)

mentioned in merge request !1063

PlasticDigits commented

2026-08-16 08:44:23 +00:00

(Migrated from gitlab.com)

mentioned in issue #528

PlasticDigits commented

2026-08-16 08:46:23 +00:00

(Migrated from gitlab.com)

added 6 commits

47729a82...f77caabb - 5 commits from branch main
781f87c5 - merge: resolve Makefile after #529 and #531 on main

Compare with previous version

added 6 commits <ul><li>47729a82...f77caabb - 5 commits from branch <code>main</code></li><li>781f87c5 - merge: resolve Makefile after #529 and #531 on main</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/346/diffs?diff_id=1964222233&start_sha=47729a82eb88cbe9238a3dad4e49b002df2e5d4c)

PlasticDigits commented

2026-08-16 08:46:26 +00:00

(Migrated from gitlab.com)

mentioned in commit a879d25890

mentioned in commit a879d25890ac67a322646c6715befc0f948c32ac