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

Increase ElevatorPulley view distance to 512 #7852

Closed
wants to merge 3 commits into from

Conversation

KCreate
Copy link

@KCreate KCreate commented Mar 8, 2025

This change increases the view distance of the elevator pulley to 512. This ensures that the elevator pulley is always visible and doesn't disappear whenever the elevator drops below 128 blocks.

This is my first interaction with the Create codebase, please tell me if this change wouldn't have the effect I'm expecting it to have.

Feedback is greatly appreciated.

WeidosOddities and others added 3 commits March 8, 2025 11:39
…g correctly while style switching

Found the issue while adding custom bogeys. The bogeys wouldn't update their lighting while switching styles, occurred with the standard bogeys also. Probably was overlooked because Create only has one style. Thanks to Roselle for solving this issue.
- Add glass pipe visual, fluid instance, and fluid mesh
- Just pipes for now, tanks need more thought as the same visual is used
  for boilers
@VoidLeech VoidLeech added the pr status: wrong target PR is targeting the wrong branch label Mar 8, 2025
Copy link

github-actions bot commented Mar 8, 2025

@KCreate, this pull request is targeting the wrong branch. Pull requests should target the branch corresponding to the earliest supported Minecraft version unless the changes are specific to code that only exists for a newer Minecraft version. Please change the target branch, resolve any merge conflicts, and leave a message here so we can continue with the process of reviewing and merging this pull request. Thanks!

@VoidLeech VoidLeech added pr type: fix PR fixes a bug area: rendering Issue or PR is related to rendering pr flag: simple PR has minimal changes labels Mar 8, 2025
@KCreate KCreate changed the base branch from mc1.21.1/dev to mc1.20.1/dev March 8, 2025 21:29
@KCreate KCreate changed the base branch from mc1.20.1/dev to mc1.21.1/dev March 8, 2025 21:30
@KCreate
Copy link
Author

KCreate commented Mar 8, 2025

Closing this and opening a new PR since I messed up the base branch.

@KCreate KCreate closed this Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: rendering Issue or PR is related to rendering pr flag: simple PR has minimal changes pr status: wrong target PR is targeting the wrong branch pr type: fix PR fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants