shivansh-bhatnagar18 / multiplayer-uno

https://multiplayer-uno.vercel.app
21 stars 43 forks source link

Frontend: Display The Player Whose Turn It Is Currently #165

Closed asmit27rai closed 6 days ago

asmit27rai commented 1 week ago

Description

  1. Brightened player icons for active turn indication.

  2. Displayed current player's name on game board.

  3. Tailwind CSS utility classes have been applied for consistent styling of the player's name.

Checklist

Screenshots

Screenshot 2024-06-27 000533

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
multiplayer-uno ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2024 6:33pm
kuv2707 commented 1 week ago

Just to be clear, we display every player's name below the cards, and in a separate place, display the name of the player currently having turn, right?

kuv2707 commented 1 week ago

Both of these things are being tracked in separate issues, and this one deals with displaying the current player's name in some central location.

asmit27rai commented 1 week ago

Just to be clear, we display every player's name below the cards, and in a separate place, display the name of the player currently having turn, right?

Yes ,here it will showing the player's name currently having turn.

asmit27rai commented 1 week ago

Both of these things are being tracked in separate issues, and this one deals with displaying the current player's name in some central location.

So should i show the name of player currently having turn at center of board or where?

kuv2707 commented 1 week ago

We should display the current player's name in some central location. All the individual players will also have their names displayed

asmit27rai commented 1 week ago

We should display the current player's name in some central location. All the individual players will also have their names displayed

OK I will update the PR accordingly.

asmit27rai commented 1 week ago

@kuv2707 Done With All The Changes.

kuv2707 commented 6 days ago

This works for now, thanks @asmit27rai