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

ENH: Implementing a Command or Config to Disable Shaking #1585

Open
TAHuntling opened this issue May 17, 2024 · 1 comment
Open

ENH: Implementing a Command or Config to Disable Shaking #1585

TAHuntling opened this issue May 17, 2024 · 1 comment
Labels
enhancement New feature or request P-Fixer This issue is confirmed, but is not prioritized to be fixed.

Comments

@TAHuntling
Copy link
Contributor

Is your feature request related to a problem?

After having a conversation with a community member, she mentioned motion sickness caused by the shaking effects in Venture Explorer as well as exploding missiles on crux. It was brought to my attention this may affect players with epilepsy as well.

Describe the solution you'd like

After messing around with changing shake effects, specifically within the AgSpaceStuff.cpp file, it is possible to disable or limit shaking effects. My proposition is to develop either a command for SlashCommandHandler.cpp which would help to disable this effect for those players who need it. /screenshake true or /screenshake false where screenshake true would be the default for all players.

Repository breaking implications

It may not be possible to configure the screenshaking effect for a single player, which if done incorrectly could potentially disable screenshaking for every player, rather than the one using the command.

Describe alternatives you've considered

No response

Additional context

This would just be a great help for those with epilepsy or motion sensitivities..

Thanks to Ginger for bringing this to my attention

@TAHuntling TAHuntling added enhancement New feature or request triage An issue that needs triage labels May 17, 2024
@EmosewaMC EmosewaMC added P-Fixer This issue is confirmed, but is not prioritized to be fixed. and removed triage An issue that needs triage labels May 17, 2024
@EmosewaMC
Copy link
Collaborator

as discussed in discord, this is not exactly going to be the easiest thing to implement. Many FX are hard coded and baked into skills and the client. this flag would not have as much of an effect as you may think and it would be a ton of work to implement for a partial implementation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P-Fixer This issue is confirmed, but is not prioritized to be fixed.
Projects
None yet
Development

No branches or pull requests

2 participants