Skip to content

Conversation

@NSK07
Copy link
Contributor

@NSK07 NSK07 commented Mar 19, 2024

Set minimum height when only Tournament winners is getting displayed in the lobby screen. This will fix #14864 .Adding screenshots before and after code change for reference:
Before :
image

After :

image

@ornicar ornicar merged commit b421bf7 into lichess-org:master Mar 19, 2024
@randompearl-stack
Copy link
Contributor

randompearl-stack commented Mar 19, 2024

This only fixes the tournament winners, the leaderboard div will still be missing.

when rating not shown
norating
when rating is shown
showrating

@NSK07
Copy link
Contributor Author

NSK07 commented Mar 19, 2024

@randompearl-stack when you turn off Show player ratings it is suppose to remove the leaderboard as well as other places where rating is showed. The issue is that when the leaderboard gets removed, Tournament Winners has very low space. This code is to fix the height issue.
image

@randompearl-stack
Copy link
Contributor

randompearl-stack commented Mar 19, 2024

Okay nevermind, the fix works, I thought that it would still show the leaderboard but without the players' ratings, like how the leaderboard page shows only the names of the top players.

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.

Tournament winners section on the home page has very low height on desktop screens when show ratings is turned off

3 participants