Skip to content

UI Improvements: Mentees & Alumni Section #151

Open
TRIPLE-ADE wants to merge 9 commits intoDxmentorship:mainfrom
TRIPLE-ADE:TRIPLE-ADE-patch-1
Open

UI Improvements: Mentees & Alumni Section #151
TRIPLE-ADE wants to merge 9 commits intoDxmentorship:mainfrom
TRIPLE-ADE:TRIPLE-ADE-patch-1

Conversation

@TRIPLE-ADE
Copy link
Copy Markdown
Contributor

UI Improvements: Mentees & Alumni Section

Changes Made

  • Improved responsiveness of the Mentees & Alumni dynamic routes for better user experience across devices.
  • Updated background styling to enhance header visibility, preventing it from blending into the white background.

Before & After

A visual demonstration of the improvements:

before.mp4
after.mp4

📌 Click the thumbnail to watch the video

Additional Notes

These updates enhance usability and responsiveness, ensuring a seamless experience for all users. 🚀


@netlify
Copy link
Copy Markdown

netlify bot commented Mar 8, 2025

👷 Deploy request for dxmentorshipv2 pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 766a4eb

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dxmentorship ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2025 1:48pm
dxmentorship-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2025 1:48pm
dxmentorship-xle4 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2025 1:48pm

@FatumaA
Copy link
Copy Markdown
Contributor

FatumaA commented Mar 9, 2025

Hi,
Just a friendly peer review... you should probably break this into 2 PRs. One that adds your details to the site and one that fixes the ui issues.

Also, seems there are related issues about your fix, you could reference them in the PR that will have your ui fixes 🙂

@TRIPLE-ADE
Copy link
Copy Markdown
Contributor Author

@FatumaA thanks for pointing that out I have another pull requests for adding my details #147

@FatumaA
Copy link
Copy Markdown
Contributor

FatumaA commented Mar 9, 2025

I'm not at my desk, so forgive me if I'm mistaken, but it seems this PR inherited commits from the other one, maybe you merged into it? If so, I suggest to cherry-pick the relevant changes into a fresh branch and opening a new PR from there

@TRIPLE-ADE
Copy link
Copy Markdown
Contributor Author

Alright would look into that thank you

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