-
-
Notifications
You must be signed in to change notification settings - Fork 440
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
Update settings section #896
Update settings section #896
Conversation
@dragonDScript |
Can you post a screenshot of it here? |
Is there anything (else) we want to prioritise here in particular? If you give me a design, I can probably implement it with css. |
I've added variables for all the colours in the gen5 figma, and ones for the defined fonts because that didn't exist for some reason, which I'm trying to stick to. I'll do the winecfg/winetricks section and checkboxes and then I'll do some work on the game specific settings, which i think will be a bit more involved because right now, it sucks. It completely covers the screen and has no back button or anything, when it's supposed to look like this: |
I'm pretty sure I've got the settings page almost completed, now the settings page for individual games needs to be fixed. |
Testing here. Noticed two things:
|
I removed the header since I was planning to make the game settings be in a modal (like in the new figma design, where it's missing as well), but won't do that in this PR because of how much it'll need to change. I'll add it back until the game settings page is changed. Yeah, I grabbed the colours from the latest figma I design (afaik), but I can change them back if you want me to. I was changing the variable names to make more sense with the figma, and it was difficult to map them over from the old names to the new ones, but I can change the colours back now if necessary. |
@wiryfuture let's keep the old Yellow color |
I want to iconify the footer buttons and then I think this will be good |
> add icons next
Yes, looks good now. We need to implement theming on the next version so we can make this more versatile. But for now, let's keep this as the default one. |
yes, send a screenshot with the result :) |
I think I've made some good variables, so it should be pretty easy now 🥳 |
@flavioislima ready for another review on this |
Can add the button style for copy to clipboard to the bottom buttons of the log page aswell? Also with text Everything else looked fine for me |
Is this ok? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks pretty good now ⚔️
Trying to follow the new Figma design as much as possible.
#768
Still got some work to do but it's a start.