-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
More than 6 banner layers #60
Conversation
"complexity" thank you codefactor for explaining what's wrong with the code really helpful |
Will this even work? Last time I checked there was a client limit on this and it refused to send packets for clicks in the gui with banners over layer limit |
It removes all the first layers before the 6 last layers in the input slot and brings them back in the output slot with the new edits |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was just a rough look-through, will check again after changes are made
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/BannerLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
src/main/java/org/purpurmc/purpurextras/modules/LoomMaxLayersModule.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a newline at the end of file and I'll be happy
yes sir 🫡 |
adds option to have more than 6 banner layers