Jump to content

Fancy Lobby Chat in Match Setup.


gentz
 Share

Recommended Posts

Shrinking the right panel by 25% and compressing the player selector thingy gives enough room for a player list! See screenshots bellow.

 

7 hours ago, mapkoc said:

Both are pretty sweet. Maybe you can toggle the view lobby chat with a hotkey or like a sidetab that expands and collapses

I'll try doing that tomorrow on my A23 build, and if it works I'll submit it to phabricator.

Screenshot_from_2018-01-02_19-43-56.png

Screenshot_from_2018-01-02_19-44-07.png

Edited by gentz
  • Like 1
  • Thanks 1
Link to comment
Share on other sites

The second chat column could be used to display which game settings changed however (that would be quite noisy. Browsing through 50 maps = 50 chat messages). This would allow players to see which settings changed (and also allow random maps to change settings without doing it behind the users back). On the other hand that might be so noisy that it's just not read.

Link to comment
Share on other sites

19 hours ago, gameboy said:

@gentz How is your test? Can it be submitted?

Just started working on it.

17 hours ago, elexis said:

The second chat column could be used to display which game settings changed however (that would be quite noisy. Browsing through 50 maps = 50 chat messages). This would allow players to see which settings changed (and also allow random maps to change settings without doing it behind the users back). On the other hand that might be so noisy that it's just not read.

Can't we instead extend the current "A setting has changed" message to be more specific, like "The map has changed"?

Edited by gentz
Link to comment
Share on other sites

I've made this little patch which shrinks the size of all the existing stuff: https://code.wildfiregames.com/D1194

It is required for the addition of a player list to the left of the player selector thingy. Its also be needed for the creation of a second chat window, unless you want it peanut sized at 1024x768.

The other option would be turning the existing chat window into an equivalently sized window with 3 tabs (2 chats and 1 player list,) however I'd imagine that wouldn't be as nice.

Edited by gentz
Link to comment
Share on other sites

Bellow is a patch with 3 things added to A22:

1- Chat panel like in the first patch

2- Player list like in second set of screenshots

3- Advanced what changed messages (I'll add it to A23 to phabricator tomorrow)

The generic something has changed message has been replaced with the following:

The map has been changed'

The map type has been changed

The map filter has been changed

he map path has beem changed

The game\'s speed has been changed

The map\'s size has been changed

The game\'s population cap has been changed

The game\'s starting resources has been changed

The length of the game\'s ceasefire has been changed

The game\'s victory condition has been changed

The teams have been changed

The total number of relics has been changed

The number of minutes till a player has won has been changed

Whether heros can garrison has been changed

Whether the game\'s map is revealed has been changed

Whether the game\'s map is explored has been changed

Whether the game has treasures has been changed

Whether the game allows spies has been changed

Whether the game\'s teams are locked has been changed

Whether the gme is last man standing or not has been changed

'Whether cheats are enabled has been changed

Whether the game is rated has been changed

The game\'s name has been changed

The game\'s description has been changed

LobbyAndAdvancesMessagesPatch.patch

Edited by gentz
Link to comment
Share on other sites

16 hours ago, gentz said:

Can't we instead extend the current "A setting has changed" message to be more specific, like "The map has changed"?

Sure, but that message is currently only printed if anyones ready state was reset. So if you browse the 60 random maps to find the best suited one and noone pressed ready, there will be at most one message rahter than 60.

It could however show a summary of the changes, so if the map selection changed 60 times without reseting anyones readystate, it could still display only the difference from previous to current map selection. (Just requires one more gamesetup rewrite which we probably would need for the "let everyone setup the game" feature too).

Link to comment
Share on other sites

In this setup with everyone being able to change things, what sort of changes are allowed?

If things like maps are allowed, wouldnt there be chaos? Perhaps it could be based on voting.

which may not workout if all 8 want different things. 

Link to comment
Share on other sites

7 hours ago, elexis said:

Sure, but that message is currently only printed if anyones ready state was reset. So if you browse the 60 random maps to find the best suited one and noone pressed ready, there will be at most one message rahter than 60.

In the patch I have it setup to print a message if someone is ready or if the change is different from the last change. So if someone cycles thought 60 maps they will have one message. I'd argue its the users responsibility to check that they like the new map before hitting ready.

Edit: Well that's sopouesed to happen, still some bugs to iron out.

Edited by gentz
Link to comment
Share on other sites

8 hours ago, (-_-) said:

In this setup with everyone being able to change things, what sort of changes are allowed?

If things like maps are allowed, wouldnt there be chaos? Perhaps it could be based on voting.

which may not workout if all 8 want different things. 

Imarok planned in #3806 that the host could chose if players can chose one of four privilege choices. Too bad the motivation has stalled. Perhaps we can scratch some motivation, it will be necessary to implement dedicated servers too (as the former gamesetup controller might leave if its not the host).

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...