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

Implement adding/ removing non-steam games from card and improve viewing games on card #1

Merged
merged 7 commits into from
Oct 27, 2023

Conversation

jessebofill
Copy link
Contributor

Ok so here's my idea for ui. you can mess with it however you want. I made the games list it's own modal that is now scrollable (more of an overlay but i think it looks nice for scorlling) and can be opened directly from the card action menu or within the edit card modal itself. In the edit card modal theres an accordian panel (can be opened/ closed) which lists all non-steam games with checkboxes allowing for quickly adding/ removing multiple games without having to popup menus and select one at a time. theres also text box to allow for filtering the list down and making it even more manageable. lastly, when making changes to the non steam games it will show how many are going to be added and deleted, and if you open the game list modal from within the editor it will also show games to be addeed in green and removals in red.

Also i added a few files for types as well as audio module that you might wanna move around. I've been trying to get some of that into dfl for a while but still waiting on that, so in the future you should just be able to import from there.

@CEbbinghaus
Copy link
Owner

Generally looks good. I will fix up any small problems

@CEbbinghaus CEbbinghaus merged commit 62b6408 into CEbbinghaus:master Oct 27, 2023
1 check passed
@CEbbinghaus CEbbinghaus added this to the 1.0.0 milestone Feb 16, 2024
@CEbbinghaus CEbbinghaus added the enhancement New feature or request label Feb 16, 2024
@CEbbinghaus CEbbinghaus self-assigned this Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants