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

Multiple loadouts UI part1 #1736

Merged
merged 23 commits into from
Jul 15, 2024
Merged

Multiple loadouts UI part1 #1736

merged 23 commits into from
Jul 15, 2024

Conversation

Al12rs
Copy link
Contributor

@Al12rs Al12rs commented Jul 9, 2024

This is a first PR of multiple to implement designs for #401
This is a work in progress.

Includes:

Figma design:
https://www.figma.com/file/TbJiArfCPTv9fQFIEPuwd5/%F0%9F%93%B1-Loadouts?type=design&node-id=67%3A3595&mode=design&t=lkBadJsMn6yv9fem-1
Can be found in #1180

@Al12rs Al12rs requested a review from erri120 July 9, 2024 10:30
@Al12rs Al12rs self-assigned this Jul 9, 2024
Copy link
Member

@erri120 erri120 left a comment

Choose a reason for hiding this comment

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

Looks good, only thing missing is the hover state on the entire card.

@Al12rs
Copy link
Contributor Author

Al12rs commented Jul 9, 2024

I'm fine with not merging this for a while to clean this up, I just pushed what I had to move on to the bugs in the meantime, but yeah I wasn't really finished with this

@Al12rs Al12rs requested a review from erri120 July 10, 2024 10:31
@Al12rs Al12rs merged commit e06c537 into main Jul 15, 2024
12 checks passed
@Al12rs Al12rs deleted the feat/multiple-loadouts-part1 branch July 15, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants