Broadcast teams: White is first #15642
Conversation
| @@ -70,16 +70,17 @@ const renderTeams = ( | |||
| cloudEval?: MultiCloudEval, | |||
| ): MaybeVNodes => | |||
| teams.table.map(row => { | |||
| const firstTeam = row.teams[0]; | |||
| const firstTeam = row.teams[1]; | |||
| const secondTeam = row.teams[0]; | |||
There was a problem hiding this comment.
This assumes that the second team is first, and the first team is second.
I doubt that it's always true, but if it was, then it should be fixed upstream in the backend, and not reverted in the frontend.
There was a problem hiding this comment.
Well... I may have done it in the frontend, but this is currently how lila is generating the teams (not just in this case at all)
There was a problem hiding this comment.
The first random team broadcast I looked at has the first players using the white pieces:
https://lichess.org/broadcast/french-cup-grand-finals-chartres-2024--womens-top-12-club-championship/semi-finals/EisjcYTg#teams
|
What do you mean by
|
*Visible board In this case, I meant: when we see teams with no visible board |
|
This PR needs to be reworked |
Screen.recording.2024-07-03.00.29.12.webm
In tournament standards the list starts with White first
as it is not directly connected to a visible tray
let's try to leave this at default