Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theseus Skin Manager #1277

Merged
merged 48 commits into from
Jan 16, 2025
Merged

Conversation

Jakobabs
Copy link

@Jakobabs Jakobabs commented Jul 9, 2024

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)

@MbtMarco
Copy link

MbtMarco commented Jul 9, 2024

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).

@Jakobabs
Copy link
Author

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

@Norbiros Norbiros mentioned this pull request Nov 30, 2024
2 tasks
@SturdyFool10
Copy link

SturdyFool10 commented Nov 30, 2024

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

@Jakobabs
Copy link
Author

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

@SturdyFool10
Copy link

SturdyFool10 commented Dec 24, 2024

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

@Prospector Prospector changed the base branch from main to app-skin-manager January 9, 2025 21:11
Prospector and others added 19 commits January 9, 2025 13:14
Co-authored-by: Jai Agrawal <[email protected]>
Co-authored-by: Prospector <[email protected]>
… are present (on the main mod page) (modrinth#3133)

* feat: Automatically open download modal when filter queries are present

* chore: Use a hash to open the modal, and make the filter queries independent of the modal
- Added min, max, and step fields to the input field of a slider
- Made the input field a number field to apply min, max, step, and accessibility
* chore: run lint

* fix: whoops messed up lint
…download modal use (modrinth#3138)

* feat: Automatically open download modal when filter queries are present

* chore: Use a hash to open the modal, and make the filter queries independent of the modal

* chore: Correct to use emit for taking functions

* chore: Add filter query and download hash to the url during modal use, and fix linting issues

* chore(frontend): Undo changes to NewModal

My computer does not like running the app, making testing a lot harder, so I'll undo this change, at least for now

* Remove extra line

---------

Signed-off-by: Prospector <[email protected]>
Co-authored-by: Prospector <[email protected]>
* feat(frontend): tags link to a search

* fix category type

* process feedback

---------

Co-authored-by: Prospector <[email protected]>
* Rough draft for fix for Mojang servers being down causing infinite installation

* Add "pack installed" install step

* Allow repairing an instance from Library to recover pack contents

* Allow repair from instance page

* Deduplicate repair code

* Fix lint

* Fix lint (for real this time)

---------

Co-authored-by: Jai Agrawal <[email protected]>
* chore: show resubscribe on `failed` status

Signed-off-by: Evan Song <[email protected]>

* chore: hide cancel on failed

Signed-off-by: Evan Song <[email protected]>

* update copy

Signed-off-by: Evan Song <[email protected]>

---------

Signed-off-by: Evan Song <[email protected]>
* Fix typescript for OverflowMenu

* Revert Discover content dropdown change to non-hoverable OverflowMenu

* Lint
# Conflicts:
#	apps/app-frontend/src/App.vue
@Prospector Prospector marked this pull request as ready for review January 16, 2025 00:30
@Prospector
Copy link
Member

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

@Prospector Prospector merged commit fc260b7 into modrinth:app-skin-manager Jan 16, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Skins page In-app skin uploading functionality