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

Don't hide the skybox in Spectator mode when inside water, lava or powder snow #2485

Closed
muzikbike opened this issue May 7, 2024 · 5 comments
Labels
E-will-not-fix Closed: Will not fix T-enhancement Type: Enhancement

Comments

@muzikbike
Copy link

Request Description

Sodium hides the sky and sun/moon if the camera is subject to water, lava or powder snow fog as a fix for https://bugs.mojang.com/browse/MC-152504. However, Spectator mode isn't intended to be "obscured" by things as the other game modes are - for example, you can see through blocks when inside them unlike in Creative/Survival, and fog distance is considerably increased when inside water or lava compared to other game modes. It would probably make sense to only have this hiding fix apply to creative/survival/adventure modes.

2024-05-07_22 43 10
2024-05-07_22 43 15
2024-05-07_22 44 55
2024-05-07_22 45 00
2024-05-07_22 45 40
2024-05-07_22 45 42

The best fix for this would probably be to disable water, lava and powder snow fog entirely in Spectator mode - that way, there'd be no obstructions at all and you'd be able to see the sky as though you were in any other block.

@muzikbike muzikbike added S-needs-triage Status: Needs triage T-enhancement Type: Enhancement labels May 7, 2024
@douira
Copy link
Contributor

douira commented May 7, 2024

Wouldn't disabling fog make it looks significantly different?

@muzikbike
Copy link
Author

Yes, but since the point of Spectator mode is to not have visual hindrances such as the suffocation overlay or on-fire effect displayed on screen I doubt this would actually matter.

@douira
Copy link
Contributor

douira commented May 7, 2024

Vanilla's spectator also doesn't remove fog

@muzikbike
Copy link
Author

True. I'm not picky on how this is fixed, if it's decided it should even be fixed at all.

@jellysquid3
Copy link
Member

The entire reason this patch exists is to deal with the way that fog interacts with the skybox. Since Minecraft doesn't fully remove the fog, even in spectator mode, it doesn't make sense to change the behavior as you've requested.

@jellysquid3 jellysquid3 closed this as not planned Won't fix, can't repro, duplicate, stale May 12, 2024
@jellysquid3 jellysquid3 added E-will-not-fix Closed: Will not fix and removed S-needs-triage Status: Needs triage labels May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-will-not-fix Closed: Will not fix T-enhancement Type: Enhancement
Development

No branches or pull requests

3 participants