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

Prevent disabling/enabling mod loaders #1536

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

anttimaki
Copy link
Collaborator

Before the changes, mod loaders could be disabled, but in most cases it didn't really do anything. Disabling e.g. BepInEx and running the game modded still loaded the mods.

Defining how each mod loader should be actually disabled isn't trivial, and the need to actually disable them is low, as user can always just choose to run the vanilla game. Therefore it was decided that just hiding the disable option for package loaders was the best approach.

This adds 0-0.1ms rendering delay for each mod. While not ideal, it doesn't affect even the bigger profiles too badly.

Before the changes, mod loaders could be disabled, but in most cases it
didn't really do anything. Disabling e.g. BepInEx and running the game
modded still loaded the mods.

Defining how each mod loader should be actually disabled isn't trivial,
and the need to actually disable them is low, as user can always just
choose to run the vanilla game. Therefore it was decided that just
hiding the disable option for package loaders was the best approach.

This adds 0-0.1ms rendering delay for each mod. While not ideal, it
doesn't affect even the bigger profiles too badly.
Copy link
Collaborator

@VilppeRiskidev VilppeRiskidev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@anttimaki anttimaki merged commit 607a9d4 into develop Nov 13, 2024
5 checks passed
@anttimaki anttimaki deleted the hide-mod-loader-disabling branch November 13, 2024 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants