Open
Conversation
…d folder with its page.tsx and page.module.css, and added a component folder with a card.tsx and card.module.css. Haven't coded the buttons in the dashboard yet
…also made them buttons #5
jliu0140
reviewed
Jan 24, 2026
Contributor
jliu0140
left a comment
jliu0140
left a comment
There was a problem hiding this comment.
The dashboard looks good! Just left some small comments to maybe discuss. Also, I think components should go under app. Otherwise, looks great!
navyaa31
requested changes
Jan 25, 2026
Collaborator
navyaa31
left a comment
navyaa31
left a comment
There was a problem hiding this comment.
Dashboard looks great! I just left a few comments, lmk if you have any questions!
…another place and the card modules which I deleted as they weren't necessary
siwenshao
reviewed
Jan 28, 2026
siwenshao
reviewed
Jan 28, 2026
siwenshao
reviewed
Jan 28, 2026
siwenshao
reviewed
Jan 29, 2026
siwenshao
approved these changes
Jan 29, 2026
siwenshao
left a comment
siwenshao
left a comment
There was a problem hiding this comment.
Everything looks pretty good! I just left a couple of small nit to fix that would be good to look into before merging.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tracking Info
Old PR Resolves #5 and #4
New PR Resolves #9
Changes
Old Changes (from old PR):
Charlie and Yuzuki created a dashboard folder inside the app folder containing a page.tsx and page.module.css. We also added a components folder inside of src containing the card.tsx, card.module.css, sidebar.tsx, and sidebar.module.css. We also added a little code snippet at the end of .github and modified the layout.tsx file inside of the app folder.
Fixes (from old PR):
I deleted the card component with its CSS file and instead used a inside of the dashboard page to code the cards. I also moved the imports to the two fonts from the CSS module they were before to the root layout.tsx file. I also deleted the png files and downladed svg files as requested. I also fixed all the minor problems listed.
New Changes (From New PR):
I made dashboard screen with the navigation bar resizable. That achieve that, I added a bunch of @media (min-width 748px) in page.modules.css inside the dashboard and sidebar modules. I added these lines to make it so that if the screen width passes 748 pixels, the dashboard goes from mobile layout to desktop layout.
Testing
Old Testing (from old PR):
We confirmed the changes worked by looking at the dashboard and ensuring that the cards would adapt to different screen sizes, and ensure that the buttons in the navigation bar and the cards were clickable.
New Testing (from new PR):
I tested the changes by pressing F12 and looking how the dashboard and navigation bar behaved on different screen sizes.
Confirmation of Change
(For old and new PR):
To see the change, you can do npm run dev on the frontend. There, you will see the navigation bar immediately. Then, when you click on the dashboard button, you will see the dashboard.