Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add info about game type/packs to room #44

Closed
tooomm opened this issue Sep 5, 2017 · 1 comment
Closed

add info about game type/packs to room #44

tooomm opened this issue Sep 5, 2017 · 1 comment

Comments

@tooomm
Copy link
Contributor

tooomm commented Sep 5, 2017

Copied over from old repo: arxanas/drafts.ninja#34

(Just had a game where somebody left after he realized that the draft was to test the new set (tripple emn draft) instead of being a perfect styled emn-emn-soi draft. Draft got destroyed in last pack.)

In general a nice information and avoids problems.
It should be displayed inside the room, as well as outside.

It could be in a new box next to settings/players. Or as some kind of headline?
Could also be removed completely once the draft starts or when it finishes to get replaced by the lands box for example.

This information used to be around and got removed in your fork.

To stress, and further clarify this issue I'll attach two screenshots:

  • view as room creator:
    host

  • view as joined player:
    joined_player

So as long as a player joins a draft over a link he has no clue what he actually joined, and what he is going to play. The host could fool around with them, or the player could easily end up in a scenario as mentioned above.
There should be a Game Information or Room Type/Settings panel for everybody. Instead a Start Game one only for the creator.

My suggestion is to change the name and content a bit (#22), make it available to all players and only display the Start Game button for the creator/admin of the room.


Linking #19 here as well.

@tooomm tooomm changed the title add info about packs to room add info about game type/packs to room Nov 18, 2017
@HerveH44
Copy link
Contributor

Fixed with PR #84

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

No branches or pull requests

2 participants