Skip to content

Polish pagination sizing and add Simic/Izzet/Gruul guild themes#150

Merged
Terraphice merged 3 commits intoindevfrom
mhh7g8-indev/fix-pagination-button-size-and-add-themes
Mar 24, 2026
Merged

Polish pagination sizing and add Simic/Izzet/Gruul guild themes#150
Terraphice merged 3 commits intoindevfrom
mhh7g8-indev/fix-pagination-button-size-and-add-themes

Conversation

@Terraphice
Copy link
Copy Markdown
Owner

@Terraphice Terraphice commented Mar 24, 2026

Motivation

  • Normalize the pagination page-number control so it visually matches the other pagination buttons.
  • Add three missing guild theme families (Simic, Izzet, Gruul) with light/dark variants so they behave like existing guild themes in the site UI.

Description

  • Changed the paginated page input to reuse the base button class by setting pageInput.className = "pagination-btn pagination-page-input" so it inherits consistent sizing and hover behavior.
  • Adjusted page-input styles in styles/gallery.scss/styles/gallery.css to remove the oversized min-height and align typography/spacing with other pagination controls.
  • Added simic, izzet, and gruul entries to THEME_FAMILIES and included them in MODE_THEME_OPTIONS in src/gallery/ui.js, wiring guild icons and resolved theme names to match the existing family implementation.
  • Added light/dark theme token blocks for simic, izzet, and gruul to styles/themes.scss and the compiled styles/themes.css, using existing background assets and swapping accent colors between light/dark modes.

Testing

  • Ran npm run lint and it completed without lint failures.
  • Ran the project smoke/e2e tests via npm test (smoke test) and all automated checks passed: 57 passed, 0 failed.

@Terraphice Terraphice merged commit 9fd509a into indev Mar 24, 2026
@Terraphice Terraphice deleted the mhh7g8-indev/fix-pagination-button-size-and-add-themes branch March 24, 2026 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant