Recreated Figma frames: login, notifications, pair, and studypals pages#2
Recreated Figma frames: login, notifications, pair, and studypals pages#2TamiOnData wants to merge 3 commits intomainfrom
Conversation
|
This is great work, however, there are differences i need you to rectify.
You did a great job Tami! After these changes, this pull request should be merged |
|
Alright i’ll make the changes
…On Sat, 12 Jul 2025 at 00:19, Posibright ***@***.***> wrote:
*Posibright* left a comment (Shadowdevsbu/prepal-frontend#2)
<#2 (comment)>
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
—
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BSMFFZCUKK3RHJKCYFFWF4T3IBA7NAVCNFSM6AAAAACA4MY4ZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTANRUGE4TEOBSGU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This pull request adds the pages for login, notifications, pair, and studypals based on the Figma designs.