Skip to content

Epic Armor Customization GUI #1215

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

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

viciscat
Copy link
Collaborator

/skyblocker custom or click the little brush icon in your inventory.

Hopefully aaron can make a DFU to port old config to the new stuff.

Video in dev showcase soon teehee

@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Mar 17, 2025
@AzureAaron AzureAaron added the new feature This issue or PR is a new feature label Mar 18, 2025
Copy link
Collaborator

@kevinthegreat1 kevinthegreat1 left a comment

Choose a reason for hiding this comment

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

There seems to be a few debug things that aren't supposed to be here.

Also, can you please send the before and after for what needs to be updated? Just the json object for before and what it needs to become after. You don't need to send the entire config file, just the part that needs to be changed.

@@ -51,8 +59,10 @@ public static void init() {
if (StackWalker.getInstance(Option.RETAIN_CLASS_REFERENCE).getCallerClass() != SkyblockerMod.class) {
throw new RuntimeException("Skyblocker: Called config init from an illegal place!");
}
System.out.println("BALLS");
Copy link
Collaborator

Choose a reason for hiding this comment

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

um

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

um uh um, that's definetly corruption, i would never write something like that (i don't remember why i put that 😭 )

@LifeIsAParadox LifeIsAParadox added changes requested This PR need changes and removed reviews needed This PR needs reviews labels Apr 8, 2025
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed changes requested This PR need changes labels Apr 15, 2025
@AzureAaron AzureAaron added merge conflicts This PR has merge conflicts that need solving. and removed reviews needed This PR needs reviews labels Apr 26, 2025
- Remake the trim buttons because mojang hates me
- Undo the OkLabColor cache
@LifeIsAParadox LifeIsAParadox added reviews needed This PR needs reviews and removed merge conflicts This PR has merge conflicts that need solving. labels Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature This issue or PR is a new feature reviews needed This PR needs reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants