code icon indicating copy to clipboard operation
code copied to clipboard

Theseus Skin Manager

Open Jakobabs opened this issue 1 year ago • 2 comments

Adds a skin manager page to the app, where users can view/change their current skin, as well as store saved skins.

image_2024-07-09_154746157 image_2024-07-09_154825415

Resolves #415 Resolves #866

(Note the input search bar is broken across the whole app since monorepo change, and is not a skin manager issue Edit: this has been fixed)

Jakobabs avatar Jul 09 '24 19:07 Jakobabs

First of all thank you, I've been waiting for this feature for a long time. I would like to suggest some improvements about your work.

Main skin manager window:

Regarding the main screen, I suggest you order "Add new skin" as the last one, in my opinion it makes more sense for it to be at the end.

Change skin window:

I suggest you leave a small empty space under "Cape" section

Also, if it is not already present, I suggest adding the possibility to change the cape without changing the skin. (Otherwise, if I only wanted to change the cape, I would have to create a new skin each time).

Lastly, I think it is redundant to use 3 different buttons for use, save and save+use. I think using the skin without saving it doesn't make much sense as the user can delete or modify it immediately afterwards anyway.

  • New skin: Cancel* | Upload (Means use+save)
  • Saved skin: Cancel* | Delete | Save | Re-Upload (Means save+use)

~~*Redundant as it is already present in the top right corner (the X could be replaced with the 'Cancel' buttom).~~

MbtMarco avatar Jul 09 '24 22:07 MbtMarco

Also, if it is not already present, I suggest adding the possibility to change the cape without changing the skin. (Otherwise, if I only wanted to change the cape, I would have to create a new skin each time).

I'll make it so the "add new skin" button autofills the current skin, so you can change the cape or arms and click upload. The cancel buttons are on all modals in the app, so removing it would go against the current modal styling

@IntMarco Thank you for the feedback

Jakobabs avatar Jul 10 '24 02:07 Jakobabs

it would also be a great feature to make the capes display what they look like, only challenge I see with that though is making it not crash if, say, Mojank releases a new cape without Modrinth needing an update

SturdyFool10 avatar Nov 30 '24 21:11 SturdyFool10

it would also be a great feature to make the capes display what they look like, only challenge I see with that though is making it not crash if, say, Mojank releases a new cape without Modrinth needing an update

Selecting a cape shows it in the 3d viewer

Jakobabs avatar Dec 15 '24 22:12 Jakobabs

absolutely awesome, all for it then, I just didn't want to be the guy who opens an enhancement a week after a feature gets added

SturdyFool10 avatar Dec 24 '24 04:12 SturdyFool10

Going to merge this into a branch on our repo to make working with it a bit easier. Thanks for your contribution!

Prospector avatar Jan 16 '25 00:01 Prospector