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

1.21.3 port #246

Draft
wants to merge 1 commit into
base: 1.22
Choose a base branch
from
Draft

1.21.3 port #246

wants to merge 1 commit into from

Conversation

lumiscosity
Copy link

@lumiscosity lumiscosity commented Nov 7, 2024

requires sisby-folk/surveyor#58, but that pr also needs fixes (accesswidener needs to be readded, version needs to be bumped)

builds and runs, but rendering is currently broken

image

help would be appreciated!

builds and runs, but rendering is completely borken; will need to look into renderlayers
@lumiscosity lumiscosity changed the title 1.12.3 port 1.21.3 port Nov 7, 2024
@lumiscosity
Copy link
Author

lumiscosity commented Nov 12, 2024

it feels like all shader color applications are shifted by one? the background and tile layers get weirdly merged and consume the one meant for the markers, the tabs use the color of the last tab for the text and the color reset for the background, etc.

i don't have nearly enough render stack experience to track this down. i slapped at the code for a few hours with no results

@sisby-folk sisby-folk changed the base branch from 1.21 to 1.22 November 13, 2024 05:36
@sisby-folk sisby-folk added the experimental hotfix ports to experimental hotfixes (won't receive modrinth release) label Nov 13, 2024
@lumiscosity
Copy link
Author

lumiscosity commented Nov 21, 2024

no luck with the render fixes, and to make matters worse, it doesn't work for 1.21.4 (tested on the release candidate) due to further JsonDataLoader changes (but hey, at least surveyor does!)

if i return to this fork i'll probably move it to 1.21.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
experimental hotfix ports to experimental hotfixes (won't receive modrinth release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants