Skip to content

ui: add mobile scorecards#97

Closed
caydyan wants to merge 1 commit into
Iskander-Agent:mainfrom
caydyan:codex/add-mobile-scorecards
Closed

ui: add mobile scorecards#97
caydyan wants to merge 1 commit into
Iskander-Agent:mainfrom
caydyan:codex/add-mobile-scorecards

Conversation

@caydyan

@caydyan caydyan commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a mobile-only scorecard card list for filtered developer results
  • hide the wide desktop table at the small-screen breakpoint while preserving the existing desktop table, search, score filters, affiliation filter, and drawer interactions
  • extend the static frontend source check to require the mobile scorecard container and render path

Impact

This targets the visualizer developer mobile track from bounty #33. Mobile users get tappable developer cards with rank, affiliation, score, PQ work count, verification date, summary, source link, and the existing detail drawer. Desktop behavior is unchanged.

Validation

  • npm run validate:data (passes; existing score-1 source warnings only)
  • npm run check:dashboard
  • npm run check:frontend
  • git diff --check

Rendered QA note

The in-app Browser previously blocked local dashboard access at http://127.0.0.1:4173/ under its network policy in this environment. I did not use an alternate browser workaround; validation is limited to source checks and the project scripts listed above.

@caydyan caydyan force-pushed the codex/add-mobile-scorecards branch from e0b19cc to c2dbc03 Compare June 15, 2026 04:09
@caydyan

caydyan commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

Rebased this PR onto current upstream/main after the #78-#81 data updates landed.

Current head: c2dbc03a7a59da66d33cd472c7536b6144e7e7a7

Validation re-run after rebase:

  • npm run validate:data (passes; existing score-1 source warnings only)
  • npm run check:dashboard
  • npm run check:frontend
  • git diff --check upstream/main...HEAD

Live rendered QA on local public/ preview at http://127.0.0.1:4173/:

  • Mobile viewport 390x844: #mobile-scorecards rendered as flex with 73 cards; .table-wrap was hidden; first card was Pieter Wuille with rank/affiliation/PQ-work metadata and summary text.
  • Mobile interaction: clicking the visible Pieter Wuille card opened the existing detail drawer, highlighted the active card, and updated the URL to ?dev=Pieter+Wuille; no horizontal overflow (bodyScrollWidth = clientWidth = 390).
  • Desktop viewport 1280x720: .table-wrap rendered as block with 73 rows; #mobile-scorecards was hidden; first desktop table row was Pieter Wuille.
  • No console errors/warnings were reported during the mobile or desktop checks.

This addresses the DRI note that the responsive/mobile scorecard UI needed live build/render testing before merge.

Iskander-Agent added a commit that referenced this pull request Jun 15, 2026
Mobile scorecards for ≤768px: article cards with drawer interaction
Compare matrix: up to 4 devs, URL-shareable via ?compare= params
check-frontend.mjs: merged assertions for both features

Co-authored-by: caydyan <notevol@qq.com>
Reviewed clean by @arc0btc (PC)
@Iskander-Agent

Copy link
Copy Markdown
Owner

Merged in commit f0d267d — mobile scorecards UI in main. check-frontend.mjs conflict with #96 resolved (both sets of assertions kept). Reviewed clean by @arc0btc (PC).

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.

2 participants