Skip to content

fix: enable core team section scroll from all pages#161

Merged
priyankarpal merged 6 commits intoreactplay:mainfrom
Sunayana-005:main
Feb 20, 2026
Merged

fix: enable core team section scroll from all pages#161
priyankarpal merged 6 commits intoreactplay:mainfrom
Sunayana-005:main

Conversation

@Sunayana-005
Copy link
Contributor

Summary

Fix the Core Team link so it scrolls correctly from any page addressing issue #153

Changes

  • Add manual scroll handling for #core-team on home page
  • Use router.push("/#core-team") when navigating from other routes

@netlify
Copy link

netlify bot commented Feb 15, 2026

Deploy Preview for reactkolkata ready!

Name Link
🔨 Latest commit 98b7087
🔍 Latest deploy log https://app.netlify.com/projects/reactkolkata/deploys/6995ed68c2d24b0007522e51
😎 Deploy Preview https://deploy-preview-161--reactkolkata.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Sunayana-005
Copy link
Contributor Author

Hey, this contribution is under Apertre 3.0

Signed-off-by: Priyankar Pal  <88102392+priyankarpal@users.noreply.github.com>
Copy link
Member

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Sunayana-005 few changes required. Attached some screenshots

  • add cursor-pointer when hover on Core Team for desktop & mobile view
  • add hover bg for Core team in mobile view just like others
  • fix alignment of Core Team in mobile view
  • add package-lock.json in .gitignore file
Image Image Image

@Sunayana-005
Copy link
Contributor Author

Changes made as requested. Please review again.

@priyankarpal
Copy link
Member

Changes made as requested. Please review again.

Hi there's a conflict

@priyankarpal priyankarpal merged commit 8931e36 into reactplay:main Feb 20, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments