Enhance Supported Skills Section with Auto-Scrolling Marquee - #257
Conversation
|
@Dushyantcoder07 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Please REVIEW it @komalharshita |
komalharshita
left a comment
There was a problem hiding this comment.
Excellent UI enhancement overall. The marquee implementation is smooth, well-scoped, and thoughtfully handled with accessibility considerations like reduced-motion fallback and hover pause behavior.
The cloned-track approach for seamless looping is implemented cleanly, and the visual refresh improves the Supported Skills section without affecting core functionality. Responsive behavior and animation polish are also handled well.
Minor observation: the PR introduces a fairly large styling refresh in addition to the marquee behavior, but the changes remain cohesive and relevant to the section being enhanced.
Approved for merge.
Summary [required]
Hey! I Added a subtle auto-scrolling marquee to the Supported Skills strip, with a duplicated track for seamless right-to-left motion, refreshed card-style styling, improved spacing and typography, and smoother hover behavior for the skill tags. It also includes a reduced-motion fallback so the section stays accessible and not distracting.
Validation passed on the touched files, with no reported errors.
Related Issue [required]
Closes #238
Type of Change [required]
data/projects.jsonWhat Was Changed [required]
| index.html | Wrapped the Supported Skills row in a marquee container and added a second track copy so the skills can scroll smoothly from right to left. |
| style.css | Reworked the Supported Skills section styling with a softer card-like surface, improved spacing and typography, pill-style skill tags, hover effects, responsive adjustments, and a reduced-motion fallback. |
| script.js | Added the client-side marquee initializer that clones the skill row once for a seamless looping animation. |
How to Test This PR [required]
git checkout Auto-Scrolling-Marquee
py -m pip install -r requirements.txt
py app.py
http://127.0.0.1:5000
It should auto-scroll smoothly from right to left, pause on hover, and stay visually consistent on different screen sizes.
py tests/test_basic.py
Expected test output:
Test Results [required]
Screenshots (if UI change)
BEFORE:

AFTER:

Self-Review Checklist [required]
feat/,fix/,docs/,data/,style/,test/python tests/test_basic.pyand all 27 tests passflake8 .locally and there are no errorsprint()orconsole.log()debug statementsNotes for Reviewer
This PR refreshes the Supported Skills strip on the home page by turning it into a subtle right-to-left marquee, improving the visual treatment to match DevPath’s design system, and adding smoother hover behavior plus responsive and reduced-motion fallbacks.