Skip to content

Conversation

@ElectricalBoy
Copy link
Collaborator

Summary

This PR:

  • extracts PlayerStats container to a separate widget
  • replaces @media rules used for adjusting stats layout with @container rules
  • adjusts surrounding stylesheets for better responsiveness

How did you test this change?

dev + browser dev tools

@ElectricalBoy ElectricalBoy self-assigned this Dec 12, 2025
@ElectricalBoy ElectricalBoy added g: valorant VALORANT c: match2 g: dota2 DOTA2 g: leagueoflegends League of Legends stylesheets Changes to stylesheets labels Dec 12, 2025
Copy link
Collaborator

@hjpalpha hjpalpha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems reasonable

@Rathoz
Copy link
Collaborator

Rathoz commented Dec 15, 2025

@FO-nTTaX do we consider that container querys have sufficient browser support?

@ElectricalBoy
Copy link
Collaborator Author

do we consider that container querys have sufficient browser support?

it's baseline since Feb 2023 according to MDN Web Docs

@FO-nTTaX
Copy link
Member

FO-nTTaX commented Dec 17, 2025

@FO-nTTaX do we consider that container querys have sufficient browser support?

yeah I think so, at least for size queries

@Eetwalt
Copy link
Collaborator

Eetwalt commented Jan 5, 2026

Whats the dev env for this?

Copy link
Collaborator

@Eetwalt Eetwalt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, tested locally for all the dev pages. Still have some questions for the alignments of the player displays and stat displays when it goes to 2 rows but that can be tackled separately. This is an improvement so merging.

@Eetwalt Eetwalt merged commit 44af625 into main Jan 7, 2026
7 checks passed
@Eetwalt Eetwalt deleted the playerstat-container branch January 7, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: match2 g: dota2 DOTA2 g: leagueoflegends League of Legends g: valorant VALORANT stylesheets Changes to stylesheets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants