Skip to content

fix: added the missing green colour from the pulsating timeline dot animation in apply page#712

Open
reach2saksham wants to merge 1 commit intoAOSSIE-Org:mainfrom
reach2saksham:fix/apply-timeline-green-missing
Open

fix: added the missing green colour from the pulsating timeline dot animation in apply page#712
reach2saksham wants to merge 1 commit intoAOSSIE-Org:mainfrom
reach2saksham:fix/apply-timeline-green-missing

Conversation

@reach2saksham
Copy link
Copy Markdown
Contributor

@reach2saksham reach2saksham commented Mar 27, 2026

Addressed Issues:

Fixes #711

Added the appropriate colour in the animation.

Screenshots/Recordings:

Before:
image

After:
image

Additional Notes:

AI Usage Disclosure:

We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.

Check one of the checkboxes below:

  • This PR does not contain AI-generated code at all.
  • This PR contains AI-generated code. I have read the AI Usage Policy and this PR complies with this policy. I have tested the code locally and I am responsible for it.

I have used the following AI models and tools: TODO

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions
  • If applicable, I have made corresponding changes or additions to the documentation
  • If applicable, I have made corresponding changes or additions to tests
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contribution Guidelines
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.
  • I have filled this PR template completely and carefully, and I understand that my PR may be closed without review otherwise.

Summary by CodeRabbit

  • Style
    • Updated the timeline visualization colors to green in light mode and yellow in dark mode for improved visual distinction.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 63a4e470-cde0-4f65-ba9e-223bc9d7a20c

📥 Commits

Reviewing files that changed from the base of the PR and between bb70285 and a1eb3ae.

📒 Files selected for processing (1)
  • src/components/apply/ApplyHeader.jsx

📝 Walkthrough

Walkthrough

Updated the animated vertical timeline in the ApplyHeader component to use green coloring (bg-[#00843D]) in light mode with a dark-mode fallback to yellow, while consolidating Framer Motion imports to remove duplicates and streamline the import structure.

Changes

Cohort / File(s) Summary
Timeline Styling & Framer Motion Imports
src/components/apply/ApplyHeader.jsx
Changed timeline fill and pulsing dot colors from yellow to green (bg-[#00843D]) with dark-mode override (dark:bg-yellow-400); consolidated Framer Motion imports by combining useInView, motion, useScroll, and useTransform into a single import statement and removed duplicate import declarations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Possibly related PRs

Suggested reviewers

  • Zahnentferner

Poem

🐰 A hop through green and dancing light,
The timeline glows, a verdant sight!
When darkness falls, the yellow gleams,
Our imports clean, our functions stream. ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding green color to the pulsating timeline dot animation on the apply page, which directly addresses the linked issue #711.
Linked Issues check ✅ Passed The code changes successfully implement the requirement from issue #711 by updating timeline colors from yellow to green (#00843D) in light mode with dark mode override, matching the specified objective.
Out of Scope Changes check ✅ Passed All changes are within scope: color updates to the timeline animation and import statement reorganization for framer-motion are directly related to the linked issue #711 requirement.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

[UI/UX] The scroll based timeline animation should be green in colour in light mode

1 participant