Skip to content

chore: move styles in Misc to relevant stylesheets#7311

Open
ElectricalBoy wants to merge 18 commits intomainfrom
misc-styles-move
Open

chore: move styles in Misc to relevant stylesheets#7311
ElectricalBoy wants to merge 18 commits intomainfrom
misc-styles-move

Conversation

@ElectricalBoy
Copy link
Copy Markdown
Collaborator

How did you test this change?

N/A; simple cut-paste

@ElectricalBoy ElectricalBoy requested review from a team March 28, 2026 01:16
@ElectricalBoy ElectricalBoy added chore stylesheets Changes to stylesheets labels Mar 28, 2026
Copy link
Copy Markdown
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.

could do some more changes i think

lines 11-18 —> Images.scss
lines 67-81 —> Icons.scss?
lines 159-267 —> new sheet? (Links.scss)
lines 323-331 —> Prizepooltable.scss
lines 377-384 —> Faction.scss
lines 386-493 —> new sheet? (Timeline.scss)
lines 539-635 —> new sheet? (Spellcard.scss)
lines 929-1051 —> remove? (for both i don't see any usage via insource search with https://tools.liquipedia.0xff0000.dev/?view=search)
lines 1111-1160 —> check for remaining usage?
lines 1257-1262 —> Faction.scss
lines 1280-1375 —> new sheet? (Placement.scss)
2062-2099 —> Icons.scss?

line numbers are based on https://github.com/Liquipedia/Lua-Modules/blob/dff7d3f85faf97b4e6daf3400c940b4c4012f770/stylesheets/commons/Miscellaneous.scss

Copy link
Copy Markdown
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.

LGTM

comments optional

Comment on lines 654 to 664
@media ( min-width: 431px ) {
.showmatch-team-mobile {
display: none;
}
}

@media ( max-width: 435px ) {
.showmatch-team-full {
display: none;
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

only usage in https://liquipedia.net/teamfortress/Template:Showmatch (and some archived stuff)
if someone wants to cvonvert those few (shit looking) old matches to SingleMatch we could kick this

Comment on lines 625 to 637
@media ( min-width: 321px ) {
.grouptable-team-mobile,
.matchlist-team-mobile {
display: none;
}
}

@media ( max-width: 320px ) {
.grouptable-team-full,
.matchlist-team-full {
display: none;
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

unused

Comment on lines 666 to 670
@media ( min-width: 300px ) {
.popover {
min-width: 275px;
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

only usage is a sandbox user space page from fo on dota2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore stylesheets Changes to stylesheets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants