-
-
Notifications
You must be signed in to change notification settings - Fork 480
[UI/UX] Library improvements #1449
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
Conversation
…her/HeroicGamesLauncher into feat/download_sidebar
is it possible to get a download queue implemented in this pr too? |
No, will be on a different one. |
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.
I added a few comments, I have to admit it's pretty hard to review all these changes in one big PR so I couldn't review it as much as I would wanted to, there's a lot going on between refactors, bugfixes, restyling, new features
First, sorry for the big PR. 😅
Some things might still change but I believe this is 99% done.
Changed
Other
ipcRenderer
on the frontend. So we should import it fromsrc/helpers
now instead ofwindow.require('electron')
Screenshots
Use the following Checklist if you have changed something on the Backend or Frontend: