ronnigaba / BoyumFoosballStats-2.0

2 stars 0 forks source link

Allow swapping player positions easily #33

Open alexandrutatarciuc opened 11 months ago

alexandrutatarciuc commented 11 months ago

We should implement a button to swap player positions easily.

I have added this as low priority for now - we should evaluate whether this is still a requirement after auto-swap has been implemented.

alexandrutatarciuc commented 11 months ago

@ronnigaba I thought about another behavior. Say I have a match Player 1 & Player 2 against Player X & Player Y. If I want to swap Player 1 with Player X I can open the dropdown for Player 1 and by choosing Player X, it will auto-swap them. So the match will become Player X & Player 2 against Player 1 & Player Y.

I think this is pretty intuitive and we don't need extra UI elements for that.

ronnigaba commented 11 months ago

Ye,a that would be pretty cool - if you select an already selected player it will swap rather than allowing multiples of the same player.

I'm also just finishing up testing on the auto-swap functionality, so we need to consider how we want to enable it - I was considering to bind it to whether a session was active or not... for now at least