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

Favorite Outfits System #439

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

Tuguicraft
Copy link

@Tuguicraft Tuguicraft commented Jul 12, 2024

I thought about adding a favorite outfits system for the change outfits menu. Initially Initially I thought to create a separate menu like the change or update outfits menu, but it was too much clutter. So I decided to make a button that pins the outfits to the top of the list, also working as a separator.
I would have loved to add a star to the left side of the outfit buttons to mark them as favorites, but the ox_lib didn't have any parameters for that and I didn't want to modify the system to that extent.
It's a bit hacky but I think it's the easiest for the user experience, as a separate menu would defeat its purpose of being accessible as it requires more clicks if you want to see your favorite and non-favorite outfits.
Another option I've thought of if you prefer the separate menu option is to give star icons to the favorite outfits in the all outfits menu. The problem is that to remove them from favorites you'll have to go to another menu, making it confusing. However, that's easy to change because it only affects the UI.

Feel free to take whatever part is usefull (mostly the backend) and modify it!

Video:
https://github.com/user-attachments/assets/260d1953-65d4-4ffc-b9ff-9ab87c6b44d2

Added a favorite outfits system for the change outfits menu, pinning them to the top of the list.
Copy link

@jaylac2000 jaylac2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants