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

Fix broken ui and update readme #2

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

telvarost
Copy link

Shaders ui is broken and will freeze the game and cause it to flicker.
Readme is outdated, so I copied the info from the discord.

Let me know if anything else needs changed. Thanks!

@mineLdiver
Copy link
Owner

Shaders screen should work fine if you're choosing the shaderpack already in a world.

@telvarost
Copy link
Author

Ah sorry my bad, you are correct. I will see if I can change this PR to either:
A) Fix the broken UI when not logged into a world
B) Remove the shaders UI option when not logged into a world

@telvarost telvarost changed the title Disable broken ui and update readme Fix broken ui and update readme Mar 18, 2024
@telvarost
Copy link
Author

Alright, I changed the PR to fix the bug rather than disable the UI, feel free to re-review :)

@telvarost
Copy link
Author

I did some testing with the built library and found it didn't apply the shader when not loaded into a world even though it did in the development environment. Therefore, I feel the better solution is to revert the selection when not loaded into a world as the fix does now. 👍

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