Skip to content

Add a table view to the award overview - #273

Open
andrzejdus wants to merge 1 commit into
pdinklag:masterfrom
andrzejdus:award-list-table
Open

andrzejdus wants to merge 1 commit into
pdinklag:masterfrom
andrzejdus:award-list-table

Conversation

@andrzejdus

@andrzejdus andrzejdus commented Sep 5, 2026

Copy link
Copy Markdown

Adds a table rendering of the award overview next to the existing card grid, with a switch between the two on the page.

  • #awards / #awards:cards — card grid (default, unchanged)
  • #awards:table — one row per award: icon, title with description, holder, record value

The renderers are split into mcstats-awardcards.js and mcstats-awardtable.js; mcstats-awardlist.js keeps the page and the switch. The switch reuses the pagination markup.

New localization keys in all languages: page.awardList.value, page.awardList.viewCards, page.awardList.viewTable.

🤖 Generated with Claude Code

The award overview was only available as a 3-per-row grid of cards, which
is hard to scan once there are a couple of hundred awards. Add a table
rendering next to it: one row per award with the icon and title (the
description underneath), the current holder, and the record value in its
own right-aligned column, using the same markup and classes as the Hall
of Fame.

The two views are switchable from the page and routed through the hash
like the player list: `#awards` and `#awards:cards` render the cards
(the default), `#awards:table` the table. The switch reuses the
pagination markup so it matches the theme without new CSS.

Each renderer lives in its own file (`mcstats-awardcards.js`,
`mcstats-awardtable.js`); `mcstats-awardlist.js` keeps only the page and
the switch.

Adds `page.awardList.value`, `page.awardList.viewCards` and
`page.awardList.viewTable` to every localization file.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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