Skip to content
New issue

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

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

Already on GitHub? # to your account

Multiplayer improvements #2760

Merged
merged 24 commits into from
Sep 16, 2021
Merged

Conversation

tritonas00
Copy link
Collaborator

@tritonas00 tritonas00 commented May 27, 2021

You can now remove your own actors and hide/unhide remote actors (useful if someone lags you) from top menu bar in multiplayer.

kkk

Also net labels remade using ImGui.

kk

Closes #2497
Fixes #2622
Fixes #479

Thanks Petr for the help!

@tritonas00 tritonas00 changed the title Added ability to remove actors in multiplayer Added ability to hide/unhide/remove actors in multiplayer Jun 10, 2021
@ohlidalp
Copy link
Member

Oops.
glitch
I'm coding a fix

@ohlidalp
Copy link
Member

As usual, this was harder than it should be. Please cherrypick the last 2: https://github.com/only-a-ptr/rigs-of-rods/commits/mp-actor-list

@ohlidalp
Copy link
Member

The only thing I found left is MP labels. While testing I noticed Unicode is completely broken - characters don't display at all. To kill 2 bugs with one blow, I made a prototype of drawing the labels with DearIMGUI.

@tritonas00 you'll have to pick from here...

  • to fix size, use the one calculated but multiply with e.g. 40. the original value is 0 - 0.6, but imgui needs it in pixels.
  • to give the text some highlight, use drawlist->AddQuad() to put darker rectangle around the text.
  • Finally, text centering needs to be fixed, currently drawn all to the right from the correct spot.

💤

@tritonas00
Copy link
Collaborator Author

The only thing I found left is MP labels

and turbojet/props

Nice one with imgui labels, will definitely check it out

@tritonas00 tritonas00 changed the title Added ability to hide/unhide/remove actors in multiplayer Multiplayer improvements Jun 29, 2021
@ohlidalp
Copy link
Member

ohlidalp commented Aug 23, 2021

obrazek
I centered and visually improved the labels.

But I also noticed a bug (exists in master) - when you join while existing player is already in vehicle, the character will not appear in the vehicle but somewhere far below the map. This was never noticed because the old label would not be visible (too small or clipped).

@ohlidalp ohlidalp force-pushed the mp-actor-list branch 2 times, most recently from 6297266 to 37c6a32 Compare August 28, 2021 21:08
@tritonas00 tritonas00 force-pushed the mp-actor-list branch 5 times, most recently from 3e4f4c9 to 9df64f4 Compare September 14, 2021 19:56
@tritonas00 tritonas00 merged commit f31d7f8 into RigsOfRods:master Sep 16, 2021
@tritonas00 tritonas00 deleted the mp-actor-list branch September 16, 2021 15:40
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants