-
-
Notifications
You must be signed in to change notification settings - Fork 481
[UX] Improve gamepage #3613
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
[UX] Improve gamepage #3613
Conversation
Signed-off-by: Kajot-dev <[email protected]>
Signed-off-by: Kajot-dev <[email protected]>
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
src/frontend/components/UI/WikiGameInfo/components/HowLongToBeat/index.scss
Show resolved
Hide resolved
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 was testing this, I found these 2 issues:
now that the 3 dots button is back on the side of the screen, the menu open in the wrong direction for RTL languages:
The old design handles the direction this opens for reference (if I remember right it's some css based on the RTL class).
then there's one more issue related to RTL languages, the tabs
MUI component seems to not support it properly and the arrows are not the right icons and one of the icons never shows up, but I don't know if this is fixable if it's an issue with MUI itself
other than those 2 issues it looks great!
O I didn't even know, that Heroic supports RTL! As of MUI tabs this issue affects all MUI tabs in heroic so I would guess that it's a global MUI issue (I found this: mui/material-ui#30207 ; it's closed but not resolved really :)). I will think of some wrapper for the |
Fixed dots menu with a simple CSS tweak. |
Build on macos-12 failed while detaching a drive, not related to the PR |
Fixed some more issues (and ofc all of these that you mentioned):
Now I think everything is working as it should |
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.
hey @Kajot-dev can you update the PR to have the latest code from main? just to be sure it's all good (there's no conflicts but shouldn't hurt to do that and I can merge this)
@arielj Pulled changes from main. Failed e2e, I think its unrelated (fails on testing a webview and I cannot reproduce the error), but you can rerun a Github job just in case |
thanks! |
This PR aims to improve UI/UX of the redesigned gamepage view
Main fixes:
TabPanel
into separate component as it was duplicated and gamepage used the same logic for the 3rd timeHowLongToBeat
not displaying properlyA little rationale for centered button
Some screenshots with comparison:
Main info tab
Before:


After:
Extra info tab
Before:


After:
Sys requirements tab
Before:


After:
TODO:
--space-*
CSS variables for more consistencyUse the following Checklist if you have changed something on the Backend or Frontend: