-
Notifications
You must be signed in to change notification settings - Fork 118
Added mockup-image #406
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
Added mockup-image #406
Conversation
@Richajaishwal0 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
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.
Please remove unnecessary file changes & keep it only as per the required feature @Richajaishwal0
@sanjay-kv
@iitzIrFan can u check it again. |
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.
Pull Request Overview
This PR enhances the UI by replacing screenshots with interactive mockups and makes structural improvements to navigation and content organization. It removes the separate leaderboard tab and consolidates contributor information into a single "Contributors" page.
Key changes include:
- Removal of standalone leaderboard functionality and integration into contributors section
- Addition of interactive mockup components for better project visualization
- Enhanced giveaway page with live leaderboard display
- Blog content cleanup and author attribution updates
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 6 comments.
Show a summary per file
File | Description |
---|---|
src/pages/dashboard/leaderboard-page.css | Adds comprehensive CSS for removed leaderboard page styling |
src/pages/dashboard/index.tsx | Removes leaderboard tab, updates navigation, and adds demo data initialization |
src/pages/dashboard/giveaway/index.tsx | Enhances giveaway page with interactive leaderboard and improved styling |
src/database/blogs/index.tsx | Updates blog entries with author information and removes one entry |
src/components/ourProjects.tsx | Replaces static screenshots with interactive mockups and live website previews |
src/components/mockup/DeveloperMockup.tsx | Creates new interactive mockup component for project showcases |
blog/spark-architecture/index.md | Removes duplicate blog content |
blog/Spark-Architecture/index.md | Removes duplicate blog content with different casing |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
@@ -0,0 +1,581 @@ | |||
/* ===== MODERN LEADERBOARD PAGE STYLING ===== */ |
Copilot
AI
Aug 21, 2025
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.
This CSS file contains 581 lines of styles for a leaderboard page that appears to be unused since the leaderboard functionality was removed from the main dashboard. Consider removing this file or documenting if it's intended for future use.
Copilot uses AI. Check for mistakes.
const history = useHistory(); | ||
const [activeTab, setActiveTab] = useState< | ||
"home" | "discuss" | "leaderboard" | "giveaway" | "contributors" | ||
"home" | "discuss" | "giveaway" | "contributors" |
Copilot
AI
Aug 21, 2025
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.
The type definition removes 'leaderboard' but the comment on line 179 still references 'leaderboard or contributors tab'. Update the comment to reflect that only contributors tab triggers leaderboard data fetching.
Copilot uses AI. Check for mistakes.
`; | ||
|
||
// Inject styles | ||
if (typeof document !== 'undefined') { |
Copilot
AI
Aug 21, 2025
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.
Injecting styles directly into the document head at runtime is not a recommended practice. Consider moving these styles to a separate CSS file or using CSS-in-JS solutions that are better integrated with the build process.
Copilot uses AI. Check for mistakes.
const getWebsiteUrl = (title: string) => { | ||
return PROJECT_URLS[title] || "https://github.com/recodehive"; | ||
}; | ||
|
Copilot
AI
Aug 21, 2025
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.
The PROJECT_URLS configuration only contains 2 entries but is used to determine whether to show live iframe previews vs static screenshots. Consider adding a more explicit configuration or documentation about which projects support live previews.
// Each project item now includes a `url` and `livePreview` property. | |
// Use `item.livePreview` to determine whether to show a live iframe preview or a static screenshot. | |
// Example: | |
// { | |
// title: "Awesome GitHub Profile", | |
// image: "...", | |
// url: "https://recodehive.github.io/awesome-github-profiles/", | |
// livePreview: true | |
// } |
Copilot uses AI. Check for mistakes.
> | ||
<motion.iframe | ||
key={activeItem} | ||
src={PROJECT_URLS[items[activeItem].title] || "about:blank"} |
Copilot
AI
Aug 21, 2025
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.
Loading external URLs in iframes without proper sandboxing restrictions could pose security risks. While sandbox="allow-scripts allow-popups allow-forms" is specified, consider if all these permissions are necessary or if additional restrictions should be applied.
Copilot uses AI. Check for mistakes.
image: "/img/blogs/01-seo-image.png", | ||
description: | ||
"An SEO backlink is created when one website links to another, and they’re extremely important for SEO. ", | ||
"An SEO backlink is created when one website links to another, and they're extremely important for SEO. ", |
Copilot
AI
Aug 21, 2025
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.
There appears to be an extra space at the end of this description string. Consider removing the trailing space for consistency.
"An SEO backlink is created when one website links to another, and they're extremely important for SEO. ", | |
"An SEO backlink is created when one website links to another, and they're extremely important for SEO.", |
Copilot uses AI. Check for mistakes.
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Description
UI was enhanced using mockup inplace of screenshot
Fixes #324
Type of Change
Changes Made
Added mockup image
Dependencies
Same as prev
Checklist
npm run build
and attached scrrenshot in this PR.Screenshot

Before:
After:

