Phobos icon indicating copy to clipboard operation
Phobos copied to clipboard

[Minor] User side laser trails switch

Open CrimRecya opened this issue 7 months ago • 5 comments

I'm not sure if CRC is needed, it's best to do a multiplayer online test.

In ra2md.ini:

[Phobos]
EnableLaserTrails=true           ; boolean

CrimRecya avatar May 20 '25 07:05 CrimRecya

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

github-actions[bot] avatar May 20 '25 07:05 github-actions[bot]

User settings like this should be made with the possibility of them being toggleable in-game in mind, even though this is not currently available. All of the existing user option toggles operate under this principle. This currently disables parsing of laser trails, not drawing.

If you only disable drawing it is a purely visual thing that does not affect game state so no additional changes to sync logic are needed.

Starkku avatar May 20 '25 08:05 Starkku

User settings like this should be made with the possibility of them being toggleable in-game in mind, even though this is not currently available. All of the existing user option toggles operate under this principle. This currently disables parsing of laser trails, not drawing.

If you only disable drawing it is a purely visual thing that does not affect game state so no additional changes to sync logic are needed.

Yes, but I just don't think anyone likes to turn on and off this in the game because those who really need to turn it off may not be able to afford the performance cost on their computer, so skipping from reading is the best performance solution.

If this solution does not meet the requirements, then forget it.

CrimRecya avatar May 20 '25 08:05 CrimRecya

Doesn't Detail Level slider accomplish the same?

Metadorius avatar May 20 '25 09:05 Metadorius

Doesn't Detail Level slider accomplish the same?

People may prefer to turn it off rather than keep that kind of "Laser".

CrimRecya avatar May 20 '25 09:05 CrimRecya

It doesn't seem like anyone here needs it, so close it.

CrimRecya avatar Jun 27 '25 08:06 CrimRecya