Skip to content

Fix unified_inventory armor tab #69

Open

Description

@Kyuhi1

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
WYI2BK9

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

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.

Activity

  1. self-assigned this
    on Apr 7, 2023
  2. added
    bugSomething isn't working
    enhancementNew feature or request
    on Apr 9, 2023
  3. squidicuzz commented on Apr 13, 2023

    @squidicuzz
    Contributor

    I think we can add a draw/z-index priority to the armor for display in the slots.. I'll look into that.

  4. self-assigned this
    on Apr 13, 2023
  5. added this to the Beta Release milestone on Apr 13, 2023
  6. squidicuzz commented on May 28, 2023

    @squidicuzz
    Contributor

    This was done, ya?

  7. squidicuzz commented on May 28, 2023

    @squidicuzz
    Contributor

    Nevermind-- We still need to do the slot priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

bugSomething isn't workingenhancementNew feature or request

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions