Skip to content

Conversation

@Utkarsh-0304
Copy link
Contributor

@Utkarsh-0304 Utkarsh-0304 commented Oct 9, 2025

Proposed change

Resolves #2385

I have:

  • Made the dots smaller
  • Made the dots align with the text title
  • Changed the colour of the vertical line and dots to grey-400
Before After
Before After

Checklist

  • I've read and followed the contributing guidelines.
  • I've run make check-test locally; all checks and tests passed.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Summary by CodeRabbit

  • New Features
    • Added two Project Timeline milestones for October 2025: OWASP Nest Sponsorship Program Expansion and Lab Level Promotion.
  • Style
    • Refreshed Project Timeline visuals: neutralized connector line color, adjusted positioning, reduced and recolored milestone dots, and simplified layout for cleaner appearance.
  • Chores
    • Standardized timeline dates to use full month names and consistent capitalization across entries.

Walkthrough

UI and content updates: About page timeline component styling adjusted (connector color/position, dot size/color/position, removed wrapper) and timeline data normalized/extended (full month names and two new October 2025 entries). No API or exported signature changes.

Changes

Cohort / File(s) Summary of Changes
About page timeline UI tweaks
frontend/src/app/about/page.tsx
Changed timeline connector color from blue-400 to gray-400, adjusted connector and dot positioning; reduced dot from 4x4 blue to 3x3 gray; removed surrounding pt-1 wrapper div. No logic changes.
Timeline data updates
frontend/src/utils/aboutData.ts
Replaced month abbreviations with full month names and standardized capitalization; added two new timeline entries for October 2025 (OWASP Nest Sponsorship Program Expansion, Lab Level Promotion); minor spacing/reordering around new entries. No API/export changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • kasya
  • arkid15r

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Out of Scope Changes Check ⚠️ Warning In addition to the UI updates specified by issue #2385, this pull request modifies aboutData.ts by expanding month abbreviations, standardizing capitalization, and adding new timeline entries—changes that fall outside the scope of the requested UI improvements. These data adjustments were not part of the issue objectives and introduce unrelated modifications. Please separate the data formatting and new entry additions into a distinct pull request or confirm that these changes are intended to accompany the UI updates to keep this PR focused on the linked issue objectives.
✅ Passed checks (4 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly identifies the change as an improvement to the about page’s project timeline section, directly reflecting the central UI adjustments made. It avoids extraneous details while clearly indicating the scope of the update. A teammate scanning the history would immediately understand the primary focus of this PR.
Linked Issues Check ✅ Passed The pull request implements all coding objectives from issue #2385 by switching the timeline line and dots to gray-400, reducing the dot size, and aligning dots with the text titles. There are no outstanding UI requirements from the linked issue left unaddressed.
Description Check ✅ Passed The description clearly outlines the UI modifications—dot size reduction, alignment adjustments, and color changes—and references the linked issue and before/after images to illustrate the update. It is directly related to the changeset and provides sufficient context without veering off-topic.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 97f5046 and d17ff16.

📒 Files selected for processing (2)
  • frontend/src/app/about/page.tsx (1 hunks)
  • frontend/src/utils/aboutData.ts (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • frontend/src/app/about/page.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Run frontend e2e tests
  • GitHub Check: Run frontend unit tests
  • GitHub Check: Run backend tests
🔇 Additional comments (2)
frontend/src/utils/aboutData.ts (2)

66-66: LGTM! Improved date format consistency.

The change from abbreviated month names to full month names (e.g., "Aug 2024" → "August 2024") improves readability and maintains consistency across all timeline entries.

Also applies to: 72-72, 78-78, 83-83, 89-89, 95-95, 101-101


114-125: Provide evidence for new October 2025 timeline entries. Include links or documentation confirming the “OWASP Nest Sponsorship Program Expansion” and “Lab Level Promotion,” and update the PR or linked issue description to reflect these content additions alongside the UI styling changes.


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.

Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
86.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@arkid15r arkid15r added this pull request to the merge queue Oct 10, 2025
Merged via the queue into OWASP:main with commit 6d9aa8d Oct 10, 2025
24 of 25 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.

Improve about page project timeline section

2 participants