-
Notifications
You must be signed in to change notification settings - Fork 62
Refactor About Page for UI Consistency and Better UX #173
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
✅ Deploy Preview for github-spy ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughThe About page was refactored to enhance its UI using the Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant AboutPage
participant FramerMotion
participant LucideReact
User->>AboutPage: Navigates to About page
AboutPage->>FramerMotion: Triggers entry animations for hero, mission, features
AboutPage->>LucideReact: Renders icons for features
AboutPage->>User: Displays animated, styled About content
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changesNo out-of-scope changes found. Possibly related issues
Suggested labels
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Thank you @AdityaB2731 for your contribution. Please make sure your PR follows https://github.com/GitMetricsLab/github_tracker/blob/main/CONTRIBUTING.md#-pull-request-guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/pages/About/About.tsx (2)
67-80: Optimize animation performance with viewport settings.The features animation implementation is solid, but consider adding viewport settings to improve performance.
<motion.div key={idx} className="p-8 bg-white dark:bg-gray-800 shadow-lg rounded-2xl hover:shadow-xl transition duration-300 text-center border border-gray-100 dark:border-gray-700" whileHover={{ scale: 1.05 }} initial={{ opacity: 0, y: 30 }} whileInView={{ opacity: 1, y: 0 }} + viewport={{ once: true, margin: "-100px" }} transition={{ delay: idx * 0.2, duration: 0.6 }} >This ensures animations only trigger once when 100px before entering the viewport, improving performance on scroll.
71-71: Consider reducing hover scale for better accessibility.The 1.05 scale on hover might be too subtle. Consider increasing to 1.08 for better visual feedback, or ensure it meets accessibility guidelines for interactive elements.
- whileHover={{ scale: 1.05 }} + whileHover={{ scale: 1.08 }}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
package.json(1 hunks)src/pages/About/About.tsx(1 hunks)
🔇 Additional comments (6)
package.json (1)
22-22: Dependency update vetted: framer-motion@^12.23.12 is the latest stableVersion 12.23.12 is current as of August 3, 2025; the caret range is appropriate and it’s correctly added to dependencies. No further changes needed.
src/pages/About/About.tsx (5)
1-3: LGTM! Clean import structure.The imports are well-organized with framer-motion for animations and specific lucide-react icons. This aligns with the PR objective of replacing emojis with consistent icons.
4-21: Well-structured features configuration.The features array provides a clean data-driven approach for the feature cards. The consistent icon styling with dark mode support is excellent.
24-24: Excellent gradient background implementation.The gradient background with dark mode support creates a modern, polished look that enhances the visual hierarchy mentioned in the PR objectives.
28-43: Smooth hero section animations implemented correctly.The staggered animations (heading first, then paragraph) create a professional entrance effect. The animation timing and easing look appropriate.
48-60: Mission section with scroll-triggered animation works well.The
whileInViewanimation ensures the mission section animates when scrolled into view, providing good performance by not animating off-screen content.
|
hey @mehul-m-prajapati i have made the required changes and created a pr kindly look on it and revert back in case of issue or help related to this pr |
|
🎉🎉 Thank you for your contribution! Your PR #173 has been merged! 🎉🎉 |
|
Hey @mehul-m-prajapati no level was assigned with this issue can you look into that please |
|
ok @AdityaB2731 |
Related Issue
Description
This PR enhances the About Us page for the GitHub Tracker project. Key improvements include:
How Has This Been Tested?
Screenshots (if applicable)
Type of Change
Summary by CodeRabbit
New Features
Style
Chores