BGWE / BoardgameWE

Backend of the Board Game Componion application.
MIT License
2 stars 0 forks source link

Display board game thumbnail in timer's list + timer page #144

Closed FServais closed 5 years ago

FServais commented 5 years ago

Here's how it looks like for the timer's list

Screenshot 2019-05-15 at 15 37 37
FServais commented 5 years ago

@waliens @hjwk @elodieburtin I took the opportunity to display the timer's info in the timer's page. There are:

But I'm not 100% convinced, what do you think?

Screenshot 2019-05-15 at 15 35 45 Screenshot 2019-05-15 at 15 35 49 Screenshot 2019-05-15 at 15 35 55
hjwk commented 5 years ago

Putting the names of the players in the header seems redundant. Only keep the boardgame and the timer type information ? Screenshot 2019-05-15 at 15 59 04

waliens commented 5 years ago

@hjwk this is for small screens that might not show the whole list of players. Therefore you would not have to scroll down to see whose turn it is. @FServais I think it is a step in the good direction (way better visually and practically than anything we have tried the other day) but I am not entirely satisfied with it. Indeed, we still cannot see the time running for the current player (even if we know whose turn it is) and time is basically the main information here :)

FServais commented 5 years ago

Mmhh, I was not completely sure that the current time was the main information needed, but I see your point.

I had many trouble getting that current time information though, I can try again.

FServais commented 5 years ago

Ok I found a way, I shouldn't work that late anymore :p

FServais commented 5 years ago
Screenshot 2019-05-15 at 16 31 55 Screenshot 2019-05-15 at 16 31 58
FServais commented 5 years ago

Screenshot 2019-05-15 at 16 34 38

hjwk commented 5 years ago

For the reload timer, could be interesting to display the reload amount

waliens commented 5 years ago

Awesome ! The cherry on the cake would be to disable time and turn information when the screen is large enough (a)

FServais commented 5 years ago

Is there something we should display instead?

FServais commented 5 years ago

@hjwk (+ @waliens) I would need the backend to send the info in the reply :)