Skip to content

Conversation

@timb8g
Copy link

@timb8g timb8g commented Dec 17, 2025

Because NEI-GTNH has broken all resource packs for NEI. I have tried to fix the patch from PureBDcraft, but recognized its not easy because we had only 6px (24px in 128x resource packs) to make a border around the recipe background due to the dynamic layout drawing.

So I thought it would be a good idea to add more space to create textures :)

First I tried to increase the Border Padding, but this one had too much dependencies for positioning of other gui elements.
So I focused on extending only the background texture in the draw function. I called it Transparency Border :)

Additionally I had done the same for the catalyst-tab to have a good alignment here.

As result the default texture looks the same as before and I was able to fix the PureBDcraft patch.
This change should help every guy who will create resources for this mod, except the guy from Faithful ^^

Before:
Bildschirmfoto 2025-12-16 um 23 23 52

After:
Bildschirmfoto 2025-12-17 um 00 34 36

@Dream-Master Dream-Master requested a review from slprime December 17, 2025 06:59
@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 17, 2025
@slprime
Copy link
Member

slprime commented Dec 17, 2025

Please use spotlessApply

@timb8g
Copy link
Author

timb8g commented Dec 18, 2025

I have already done the /gradlew spotlessApply.
Something else I need to do?

@slprime
Copy link
Member

slprime commented Dec 20, 2025

I have already done the /gradlew spotlessApply. Something else I need to do?

try it again because problem is still here

image

@Dream-Master Dream-Master merged commit 454b4f7 into GTNewHorizons:master Dec 28, 2025
1 check passed
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 28, 2025
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.

3 participants