General model/texture improvements#87
General model/texture improvements#87testerpester2 wants to merge 30 commits intojlortiz0:fabric-1.21.4from
Conversation
- Clickers now have more detail and aren't huge - Collars now have custom inventory texture (will make it show up correctly in-game in the next commit) - collar_locker.png is now 16x - Deed of Ownerships now no longer have mixels and has been made to look more like paper, plus is now 16x - Golden Spatula now looks more inline with the other golden tools - Paws now have custom inventory texture (will make it show up correctly in-game in the next commit)
…lars into fabric-1.21.4
src/main/java/org/jlortiz/playercollars/datagen/ModelDataGenerator.java
Outdated
Show resolved
Hide resolved
src/main/resources/assets/playercollars/models/item/collar.json
Outdated
Show resolved
Hide resolved
src/main/resources/assets/playercollars/models/item/invisible_fence.json
Show resolved
Hide resolved
There was a problem hiding this comment.
is the speaker grill strip supposed to always be white? also, the sides of the button being light-colored can look strange if the clicker is dyed a dark color.
| { | ||
| "type": "minecraft:map_color", | ||
| "default": 3361970 | ||
| "when": "gui", |
There was a problem hiding this comment.
just gui means the 3d model is also used when the item is held in the hand. is that intentional, or did you only want it to use a 3d model on the head?
There was a problem hiding this comment.
this doesn't really represent the item. i prefer the 3d glass model.
There was a problem hiding this comment.
I can agree on this, it was Testers idea and I just made it look better
There was a problem hiding this comment.
It's updated to better match the 3D model now
src/main/resources/assets/playercollars/textures/item/dog_bowl.png
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
kinda difficult to tell what this is supposed to be at this resolution
There was a problem hiding this comment.
It's ment to be the paw, at this resolution was removed due to it looking weird when bigger
Co-authored-by: sorgyfrog <203951971+sorgyfrog@users.noreply.github.com>
|
I didn't mean to to request review honestly I didn't know what it did I'm not super versed in Github stuff still |
|
i'll unrequest myself for now, since there seems to be some feedback not addressed. add me for review again or ping me when you're ready. |
Tester has been problems with generated model stuff so she kinda slowed down alittle |
|
I tried this look and honestly... I really don't like the 3D model of the Paws... It doesn't look good. I prefer the old design as it looks really nice and fits the pet. I like the texture of the paws in the inventory tho! As for the Collar, that is a nice touch, seems fine. For the dog bowl, the food looks stunning! It really looks like a dog food, which is amazing and would really advice to have that in the mod! For the collar, if the mod would get Bell Collars, it could be a really confusing texture as it kinda looks like a Bell Collar. Spatula is fine, the Deed... I agree with Sorgyfrog. Their texture looks better. As for the Clicker, I suppose that looks fine, more realistic. Overall, There are things I like and things I don't like. But Good Job! :) |

Models
Changed:
Updated:
Textures
Changed:
Added:
All the textures except for the tag_overlay were made by MiffenKop!!!