-
Notifications
You must be signed in to change notification settings - Fork 814
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
Increase Max Players to 8 #7663
base: master
Are you sure you want to change the base?
Conversation
Maybe it's worth adding a display of player nicknames above them for small screen resolutions and some shapes with the appropriate player color for larger resolutions? And add the appropriate key binding to switch the display. |
I think that's what @kphoenix137 meant when he mentioned: "This could be resolved with a future QoL addition of a Party panel, along the lines of what Diablo 2 offers" |
Replace MAXVISION with MAX_PLRS Update base.cpp MAXVISION again? Add 8 player functionality Remove MAXPORTAL Update Update Update Update portal.cpp Add fixed monster tags Update automap.cpp Adjust portal positions Update automap.cpp Use default 200 max monsters Fix errors Fix mistakes
c97929f
to
e240051
Compare
I think the most intuitive solution (a solution that isn't really in the scope of this PR) would be recoloring each player based on their slot number. This may be controversial however and the proper agreed upon change should be discussed at a later time. |
@@ -62,7 +62,7 @@ void loopback::SNetGetProviderCaps(struct _SNETCAPS *caps) | |||
caps->flags = 0; // unused | |||
caps->maxmessagesize = 512; // capped to 512; underflow if < 24 | |||
caps->maxqueuesize = 0; // unused | |||
caps->maxplayers = MAX_PLRS; // capped to 4 | |||
caps->maxplayers = MaxPlayers; // capped to 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there any reason this shouldn't be capped to 1?
@kphoenix137 very minor suggestion, is it ok to change the first 4 colors a little? The orange and red are too close to each other, maybe you could do something like: It could be very helpful for colorblind people 🙋 |
I would prefer to wait for official color-blind accessible changes to be applied. I think immediately changing orange to a light red would make it more difficult to differentiate for non-color blind people as we'd be using 3 different shades of red. I think a future PR that adds toggle options for different types of color blindness that globally change the palette would be great. |
Allows for 8 players in a multiplayer game.
Details:
Exp tag icons for players 1-8:
Players 1-8 on the automap:
Players 1-8 highlight:
Potential issues: