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

Virtual Arsenal Manager #285

Draft
wants to merge 141 commits into
base: master
Choose a base branch
from
Draft

Virtual Arsenal Manager #285

wants to merge 141 commits into from

Conversation

R3voA3
Copy link
Owner

@R3voA3 R3voA3 commented Sep 26, 2023

This pull request/branch will be merged once the Virtual Arsenal Manager is ready for the public.

To-do:

  • Translation
  • UI polishing. Moving to pixelGrid and making sure UI works on all resolutions/ui sizes
  • Cleaning up functions

@R3voA3 R3voA3 added the WIP Working on it label Sep 26, 2023
@R3voA3
Copy link
Owner Author

R3voA3 commented Sep 26, 2023

20230926235340_1

Updated Presets UI.

@linkion
Copy link
Contributor

linkion commented Sep 28, 2023

I think we should remove the stats for weapons

Don't think it's really needed

@linkion
Copy link
Contributor

linkion commented Sep 28, 2023

Maybe that space can be used to have the icon and compatible items to select

┌─────────────────────────────┬────────────────────────────┐
│ MAIN TV Tree                │ Selected Item Figure       │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             ├────────────────────────────┤
│                             │ Selected item Accessories  │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
│                             │                            │
└─────────────────────────────┴────────────────────────────┘

@linkion
Copy link
Contributor

linkion commented Sep 28, 2023

Currently, when a weapon is selected, the main TV is minimized to show the Accessory TV, which is jarring.

So I'm probably just gonna change that.

@R3voA3
Copy link
Owner Author

R3voA3 commented Sep 28, 2023

I have already started with a redesign.

20230928223237_1

@linkion
Copy link
Contributor

linkion commented Sep 28, 2023

Ok, but I think that they should be split horizontally. The tv tree doesn't take that much space width wise, but it is very vertical and requires a lot of scrolling when you expand something.

@R3voA3
Copy link
Owner Author

R3voA3 commented Sep 28, 2023

Final update for tonight. @linkion

20230928235416_1

@linkion
Copy link
Contributor

linkion commented Sep 28, 2023

Nvm, that looks good

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

Successfully merging this pull request may close these issues.

5 participants