Gregtech Texture port from 1.7.x to 1.12.x

Soartex-Modded / Modded-1.12.x

Soartex Mod Textures for Minecraft 1.12.x! Download here:

https://soartex.net/downloads/mods/

27 stars 21 forks source link

Gregtech Texture port from 1.7.x to 1.12.x #25

Closed daugvinasr closed 3 years ago

artdude543 commented 5 years ago

The modId is incorrect needs to be all lowercase as per specification.
The mod folder should be GregTech to match the mod naming convention.
All ingots/dusts/gears and generic resources need to be using soartex_base as the location for the textures to prevent un-needed duplication.

I have been working on creating all the missing base textures to add to soartex_base as we have re-done all the colours/scales for 1.12.x. So until I have completed that part this PR will need to be on hold. Until the base is ready and the first two changes are performed.

But amazing work on the PR/Port! 1.12.x is slow for us as we're revamping and cleaning up all the mess from previous versions.

daugvinasr commented 5 years ago

Hi. I am quite new to this. I am not sure about soartex_base with GTCE. GTCE automatically colors ingots in the game and the pack creator has to only provide black and white texture.

daugvinasr commented 5 years ago

Turns out at first I managed to push wrong textures. Good catch. Sorry about that.

daugvinasr commented 5 years ago

I am done porting from 1.7.x. If there is any changes that you need from my end feel free to message me.

leagris commented 4 years ago

I am done porting from 1.7.x. If there is any changes that you need from my end feel free to message me.

@Ormsher Your commit still contains wrong GregTech directory name. Your repository has tow folders, one Gregtech and one GregTech. Please fix this (see @artdude543 requested changes)

LAGIdiot commented 3 years ago

Is this still a live? Is there anything that I can help with because I would like to see this finished.

Shoeboxam commented 3 years ago

I went ahead and merged this PR. Seems that the outstanding issue here is items that are not pulled from base. Perhaps this can be done post-merge if folks are really committed to it. I think the damage of not merging the PR for this long is much greater than potentially accepting some non-base textures. The folder naming was an easy fix to make afterwards. Sorry for the long process @Ormsher.

@LAGIdiot Long time no see. Nice to see you around. There's a Discord now ~~

LAGIdiot commented 3 years ago

@Shoeboxam It is nice that you remembered old me and sure I will come to hang out on Discord!