Skip to content

add ScrollProgressBar component#190

Merged
melvincwng merged 5 commits intoyfosp:mainfrom
shubsolos19:main
Mar 18, 2026
Merged

add ScrollProgressBar component#190
melvincwng merged 5 commits intoyfosp:mainfrom
shubsolos19:main

Conversation

@shubsolos19
Copy link
Copy Markdown
Contributor

What this PR does

Added a reusable ScrollProgressBar component that shows a thin
progress bar at the top of the page as the user scrolls down.

Changes made

  • Added components/ScrollProgressBar.tsx
  • Integrated it in pages/_app.tsx so it works on all pages

Tech used

  • React hooks (useState, useEffect)
  • TypeScript
  • CSS-in-JS (inline styles)

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 18, 2026

Deploy Preview for yfosp canceled.

Name Link
🔨 Latest commit 6847a04
🔍 Latest deploy log https://app.netlify.com/projects/yfosp/deploys/69bad7d719869e0008ad664f

Copy link
Copy Markdown
Member

@melvincwng melvincwng left a comment

Choose a reason for hiding this comment

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

@shubsolos19 , thanks for the self-initiated/suggested feature enhancement to our community website.

Overall logic looks good, LGTM

Before I approve and merge the PR, please add yourself as a contributor in these 2 files:

  1. contributors.tsx
  2. CONTRIBUTORS.md

Tag/ping me once that's done

@melvincwng
Copy link
Copy Markdown
Member

melvincwng commented Mar 18, 2026

@shubsolos19, add yourself as a contributor here:

Make a new table row, each table row only has 6 cells - maintain the same consistency

  1. https://github.com/yfosp/main-website/blob/main/community-website/pages/contributors.tsx
  2. https://github.com/yfosp/main-website/blob/main/CONTRIBUTORS.md

@melvincwng
Copy link
Copy Markdown
Member

Tag me once its done so I can approve and merge your request @melvincwng

@shubsolos19
Copy link
Copy Markdown
Contributor Author

@melvincwng Done! Added myself to both contributors.tsx and CONTRIBUTORS.md ✅

Copy link
Copy Markdown
Member

@melvincwng melvincwng left a comment

Choose a reason for hiding this comment

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

LGTM

@melvincwng melvincwng merged commit fcdcd04 into yfosp:main Mar 18, 2026
4 checks passed
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