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][compat] iris shaders #173

Merged
merged 2 commits into from Aug 27, 2022
Merged

[fix][compat] iris shaders #173

merged 2 commits into from Aug 27, 2022

Conversation

lonefelidae16
Copy link
Contributor

fixes #172

Use the constructor to set fields renderBackground and renderHorizontalShadows that they can be changed by other MOD. Please check the changes below:

  • EntryListWidgetMixin.java
    • methods doNotDrawBackground(EntryListWidget) and doNotDrawBars(EntryListWidget) were removed
    • new method bedrockify$ctor was added which targeted to constructor of EntryListWidget

* `EntryListWidgetMixin.java`
  - now use constructor to set `renderBackground` and `renderHorizontalShadows`
Copy link
Contributor Author

@lonefelidae16 lonefelidae16 left a comment

Choose a reason for hiding this comment

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

I read over my code and had a thought, so I’ll review it myself.

* `EntryListWidgetMixin.java`
  - use Shadow annotation to get the original methods instead of casting `this`
  - use `RETURN` in At annotation of constructor
  - unnecessary import statement
@lonefelidae16
Copy link
Contributor Author

update: for EntryListWidgetMixin.java

  • use Shadow annotation to get the original methods instead of casting this
  • use RETURN in At annotation of constructor
  • deleted unnecessary import statement

@juancarloscp52
Copy link
Owner

Looks good! Thanks!

@juancarloscp52 juancarloscp52 merged commit 8611a5d into juancarloscp52:1.19 Aug 27, 2022
@lonefelidae16 lonefelidae16 deleted the fix/compat-iris branch August 27, 2022 11:10
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.

In-game Iris menu loads with dirt background.
2 participants