CryptechTest / CryptechTest/cryptech

Fix unified_inventory armor tab

Open
#69 3 comments 0 reactions 2 assignees Claimed by @squidicuzz View on GitHub
bug enhancement
Dominant language
Lua
Stars
3
Forks
4
PR merge metrics
No merged PRs in 30d

Description

When equipping armor pieces which textures overlap in the preview window the matter you put them in matters since preview is rendered in the order grid window is set up. Therefore moving armor pieces around changes the preview render as seen in the images below.

![HByNo7d](https://user-images.githubusercontent.com/64851536/230613087-e89c4e6b-0aa7-4dbb-b5cc-aa67eee3ab8d.png)
![WYI2BK9](https://user-images.githubusercontent.com/64851536/230613101-b8dc838d-21c0-4590-afa9-ee08091b0d3f.png)

Armor is rendered based on the order seen in the image below where the ones that come next will overlap in the preview over the previous armor pieces.

![eMgKrRs](https://user-images.githubusercontent.com/64851536/230615447-394a727d-904b-41f1-9c2d-4fb396f58b60.png)

There are 2 solutions:

- Either make a grid that can accept certain items only in certain slots like the way minecraft does (helmet socket, chestplate socket..)
- Or determine an order in which these armor pieces are previewed that logically makes sense (1st leggings, 2nd chestplate, 3rd boots, 4th helmet, etc...)

Realistically only issue so far is with leggings as they should go under chestplate and boots so always rendering leggings on top would be good too.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.