Skip to content

Recreated Figma frames: login, notifications, pair, and studypals pages#2

Open
TamiOnData wants to merge 3 commits intomainfrom
tami-dashboard
Open

Recreated Figma frames: login, notifications, pair, and studypals pages#2
TamiOnData wants to merge 3 commits intomainfrom
tami-dashboard

Conversation

@TamiOnData
Copy link

This pull request adds the pages for login, notifications, pair, and studypals based on the Figma designs.

@TamiOnData TamiOnData requested a review from Posibright July 9, 2025 06:50
@Posibright
Copy link
Collaborator

This is great work, however, there are differences i need you to rectify.

  1. You changed the look if the sidebar, it would be best if you maintained the colors and sizing of the sidebar that i created (including the highlight when hovering and clicking on the sidebar being darker rather than lighter.) You can replicate the sidebar i have. This should be done on all pages you made.

  2. After that change, kindly make sure that the sidebar routes to the desired pages when clicked. Right now it doesn't and that breaks the functionality.

  3. Remove the create plan button from the pair page as well. On this page, rectify the notification size, it is different from the other page you made.

  4. In the studypals page, remove that border in the searchbar that appears when it is clicked.

  5. One more thing, in the next commit i make, the user's profile picture will be "ps.png" make that change in your work to maintain consistency.

You did a great job Tami! After these changes, this pull request should be merged

@TamiOnData
Copy link
Author

TamiOnData commented Jul 12, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants