-
Notifications
You must be signed in to change notification settings - Fork 507
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
dashboard page and layout #94
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @webdevcody on Vercel. @webdevcody first needs to authorize it. |
@@ -148,11 +148,11 @@ export default function DashboardPage() { | |||
</Card> | |||
</div> | |||
<div className="flex max-[850px]:flex-col justify-center items-center w-screen min-[850px]:h-[55vh] gap-4 m-4"> | |||
<Card className="w-[55vw] max-[850px]:w-[100vw] min-[850px]:h-[55vh] mr-4 p-4"> | |||
<Card className="w-[50vw] max-[850px]:w-[100vw] min-[850px]:h-[55vh] mr-4 p-4 border-none"> |
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.
Border none doesnt give a clear indication of how big the card is.
@@ -213,7 +213,7 @@ export default function DashboardPage() { | |||
</Button> | |||
</div> | |||
</Card> | |||
<Card className="w-[40vw] max-[850px]:w-screen min-[850px]:h-[55vh] mr-4"> | |||
<Card className="w-[42vw] max-[850px]:w-screen min-[850px]:h-[50vh] mr-4 border-none"> |
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.
55vh looks better imo since the recent games card has 55vh as height.
title: Issue #93
I fixed the dashboard page, I added a layout for the page and corrected footer and horiznotal scroll. I also fixed bottom tables, since they also had overflowed content and didn't have good UI.
Discord Username: @luka#10
What type of PR is this? (select all that apply)
Description
Related Tickets & Documents
QA Instructions, Screenshots, Recordings
UI accessibility concerns?
Added/updated tests?
[optional] Are there any post deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?