Skip to content

fix: shrink region-grid + hide extra ledger cols on mobile - #243

Merged
Flotapponnier merged 1 commit into
devfrom
fix/mobile-overflow-region-grid-and-ledger-extra-cols
Jun 1, 2026
Merged

fix: shrink region-grid + hide extra ledger cols on mobile#243
Flotapponnier merged 1 commit into
devfrom
fix/mobile-overflow-region-grid-and-ledger-extra-cols

Conversation

@Flotapponnier

Copy link
Copy Markdown
Collaborator

Le bug bench-spécifique

L'utilisateur voyait l'overflow sur `aggregator-head-lag` mais pas sur des benchs plus simples. Cause : ce bench (Solana, geo-distribué) rend deux composants extra-larges qui n'avaient pas de version mobile :

1. `region-grid.tsx` — cellules trop rigides

Avant : `grid-cols-[1fr_5rem]` partout + Product col `w-32` (128px) + gap-3.
Avec 3 régions : intrinsèque ~404px > 375px (iPhone SE). Sortait du viewport.

Fix mobile :

  • Product col : `w-32` -> `w-20` (80px)
  • Cell grid : `grid-cols-[1fr_5rem]` -> `grid-cols-[1fr_auto]` (valeur prend juste sa largeur de contenu)
  • gap-3 -> gap-1.5, px-2 -> px-1, font-size 12 -> 11

Tout repasse en densité originale dès `sm:`.

2. `ledger-table.tsx` — Slot delta et Secondary visibles sur mobile

Les colonnes p90/p99/Mean/Δ/Success étaient déjà `hidden md:table-cell`. Mais Slot delta et Secondary (qui n'apparaissent QUE sur certains benchs comme head-lag) n'avaient pas ce hide. Ajout du `hidden md:table-cell` cohérent.

Test plan

  • iPhone Safari sur /benchmarks/aggregator-head-lag
  • Region grid : 3 régions tiennent sur 375px sans débordement ni scroll horizontal interne
  • Ledger : seulement Name + p50 + Trend visibles sur mobile (pas de slot delta)
  • Desktop md+ : tables inchangées visuellement

aggregator-head-lag (and other geo-distributed Solana benches) showed
overflow on iPhone because:

- region-grid cells used grid-cols-[1fr_5rem] on every viewport; with
  3 regions + a 128px Product column the table intrinsic min was ~404px
  > 375px iPhone SE viewport. Now: grid-cols-[1fr_auto] on mobile,
  Product col w-20 (80px) instead of w-32, tighter gap, smaller value
  font. Stays at the original densities from sm: up.
- ledger Slot delta and Secondary columns were visible on mobile (no
  hidden md:table-cell), adding 2 extra mono columns to benches with
  hasSlots/secondary. Now hidden on mobile like the other aggregate
  columns (p90/p99/Mean/Delta/Success).
@Flotapponnier
Flotapponnier merged commit e2dbf99 into dev Jun 1, 2026
1 check failed
@Flotapponnier
Flotapponnier deleted the fix/mobile-overflow-region-grid-and-ledger-extra-cols branch June 9, 2026 21:42
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
…ast (#2120)

* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
…ms, Chainstack 139ms (#2122)

* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
…obinhood (#2124)

* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads

* bump cache v56/v23 to flush keyed-rpc bench set changes
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads

* bump cache v56/v23 to flush keyed-rpc bench set changes

* drop rpc-keyed-latency bench
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads

* bump cache v56/v23 to flush keyed-rpc bench set changes

* drop rpc-keyed-latency bench

* bump cache v57/v24 to flush rpc-keyed-latency drop
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* feat(robinhood): add QuickNode to keyed-rpc-robinhood bench (#243)

* fix(keyed-rpc-robinhood): update findings — Alchemy leads overall, Chainstack wins us-east

* fix(ws-head-latency-robinhood): update findings — QuickNode leads at 0.27ms, Alchemy 40ms, Chainstack 139ms

* fix: bump cache keys v55/v22 to flush stale miss for ws-head-latency-robinhood

* fix: remove em-dashes from ws-head-latency-robinhood findings (noAiDashes Zod)

* drop ws-head-latency-robinhood bench (244)

* drop all non-robinhood keyed-rpc benches

* keyed-rpc-robinhood: US-only bench, Chainstack leads

* bump cache v56/v23 to flush keyed-rpc bench set changes

* drop rpc-keyed-latency bench

* bump cache v57/v24 to flush rpc-keyed-latency drop

* add us-west to region enum in spec-schema
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads
Flotapponnier added a commit that referenced this pull request Aug 27, 2026
* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap (#2159)

* feat(rpc): add Litecoin, Babylon, Acala, Interlay benches (#246-249) (#2166)
Flotapponnier added a commit that referenced this pull request Aug 28, 2026
* fix(ci): smoke against deploy URL, not CDN-cached openchainbench.com

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap

* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix(ci): add checkHost arg to smoke — sitemap from deploy URL, pages on prod

* fix(sitemap): filter removed benches, HL spec slugs, add zerion/routescan to removed products

* fix(sitemap): isHlBuilderSlug filter + exclude REMOVED_BENCH_SLUGS from categories

* sync: merge dev into main (benches 246-249 + recent fixes) (#2167)

* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap (#2159)

* feat(rpc): add Litecoin, Babylon, Acala, Interlay benches (#246-249) (#2166)

* fix(rpc): remove em dashes from acala/babylon/interlay seo_intro (#2170)

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* fix(rpc): remove em dashes from acala/babylon/interlay seo_intro

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* fix: use query params for shareable URL instead of broken path format
Flotapponnier added a commit that referenced this pull request Aug 28, 2026
* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap (#2159)

* feat(rpc): add Litecoin, Babylon, Acala, Interlay benches (#246-249) (#2166)

* fix(rpc): remove em dashes from acala/babylon/interlay bench YAMLs (#2168)

* feat(rpc): add Litecoin, Babylon, Acala, Interlay benches (#246-249)

* fix(rpc): remove em dashes from seo_intro in acala/babylon/interlay YAMLs

* fix: direction-aware funding projection for all venues (#2163)

* fix(ci): smoke against deploy URL, not CDN-cached openchainbench.com

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap

* fix(ci): add checkHost arg to smoke — sitemap from deploy URL, pages on prod

* fix: direction-aware funding projection for all venues

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* fix: correct Robinhood Chain launch date and remove unverified provider claim (#2174)

* feat: pre-fill wallet from URL params on fee-compare (#2176)

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* feat: pre-fill wallet/venues from URL search params on fee-compare page

* fix: shareable URL uses query params not broken path format (#2177)

* fix(ci): smoke against deploy URL, not CDN-cached openchainbench.com

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap

* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix(ci): add checkHost arg to smoke — sitemap from deploy URL, pages on prod

* fix(sitemap): filter removed benches, HL spec slugs, add zerion/routescan to removed products

* fix(sitemap): isHlBuilderSlug filter + exclude REMOVED_BENCH_SLUGS from categories

* sync: merge dev into main (benches 246-249 + recent fixes) (#2167)

* fix: restore chainstack findings, new key live on both regions

* fix: us-east only bench, drop us-west dimension and service

* feat: add eu-west + sgp regions, chainstack wins all 3

* feat: GMX v2 funding rates via DataStore OI imbalance (#2153)

* feat: bench #243 keyed-rpc-robinhood — 3-region live data (#2154)

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin robinhood bench queries to region=us-east

* remove non-functional eu/sgp Railway scrape targets from Railway Prom (VPS harnesses take over)

* bump cache keys v62/v29 for eu-west+sgp VPS harness data

* fix(keyed-rpc-robinhood): Alchemy leads global avg + eu-west/sgp, Chainstack wins us-east

* revert: us-east only, no dimensions, Chainstack leads

* fix: pin queries to region=us-east to ignore stale eu/sgp prom data

* fix: augment HL position reconstruction with clearinghouseState to bypass 2000-fill cap (#2159)

* feat(rpc): add Litecoin, Babylon, Acala, Interlay benches (#246-249) (#2166)

* fix(rpc): remove em dashes from acala/babylon/interlay seo_intro (#2170)

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* fix(rpc): remove em dashes from acala/babylon/interlay seo_intro

* fix(rpc): replace dead Acala/Interlay providers with live endpoints

* fix: use query params for shareable URL instead of broken path format

* fix: split Gains taker vs vault fee, add per-trade equiv fee column (#2178)
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