Conversation
…nd add GitHub showcase
…sShowcase for improved structure and functionality
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
There was a problem hiding this comment.
Pull request overview
Bumps the app to v6.0.3 while simplifying the media/library area and refactoring projects UI (including adding a GitHub activity section) alongside an expanded Reem chatbot system prompt.
Changes:
- Version bump to 6.0.3 and minor UI tweaks (footer version color, GitHub section spacing, default theme).
- Remove library/media dashboard & quick actions routes/components; update navigation/breadcrumbs away from
/media//library. - Refactor Projects showcase into smaller components (ProjectCard/ProjectsFilter) and add GitHub showcase to the Projects page.
Reviewed changes
Copilot reviewed 34 out of 34 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Bumps package version to 6.0.3 |
| README.md | Updates displayed version to 6.0.3 |
| data/main/footerLinks.ts | Removes Library link from footer |
| data/main/chatbot-system-prompt.ts | Expands/rewrites Reem system prompt + config traits/capabilities |
| data/hubs/libraryData.ts | Removes library dataset (file deleted) |
| components/quickActions/quickActions.tsx | Removes quick actions component (file deleted) |
| components/quickActions/index.ts | Removes quick actions barrel export (file deleted) |
| components/projects/projects-showcase.utils.ts | Extends license badge/emoji mappings; minor formatting |
| components/projects/index.ts | Exposes ProjectCard/ProjectsFilter exports |
| components/projects/ProjectsShowcase.tsx | Splits UI into ProjectsFilter + ProjectCard; adjusts grid |
| components/projects/ProjectsFilter.tsx | New component for category filter buttons |
| components/projects/ProjectCard.tsx | New extracted ProjectCard component |
| components/nav/navbarItems.tsx | Changes nav item from “Hub” (/media) to “Blogs” (/blog) |
| components/mediaDashboard/index.ts | Removes media dashboard export (file deleted) |
| components/mediaDashboard/dashboard.tsx | Removes media dashboard page component (file deleted) |
| components/library/shelfCard.tsx | Removes library card UI (file deleted) |
| components/library/othersShelf.tsx | Removes library “Others” shelf (file deleted) |
| components/library/index.ts | Removes library barrel export (file deleted) |
| components/library/booksShelf.tsx | Removes library “Books” shelf (file deleted) |
| components/github/GitHubShowcase.tsx | Tweaks spacing for GitHub section |
| components/footer/Footer.tsx | Changes version display styling |
| components/chatbot/ChatBot.tsx | Changes consent overlay behavior and scroll locking |
| components/blog/BlogBreadcrumb.tsx | Breadcrumb now points to /blog and renames “Media” → “Blogs” |
| components/about/index.ts | Removes about barrel export (file deleted) |
| components/about/AboutComponents.tsx | Removes about components file (file deleted) |
| app/media/page.tsx | Removes /media page (file deleted) |
| app/media/loading.tsx | Removes /media loading state (file deleted) |
| app/media/layout.tsx | Removes /media layout/metadata (file deleted) |
| app/layout.tsx | Sets default theme to dark |
| app/(media)/projects/page.tsx | Adds GitHub showcase section to Projects page |
| app/(media)/library/page.tsx | Removes /library page (file deleted) |
| app/(media)/library/loading.tsx | Removes /library loading state (file deleted) |
| app/(media)/layout.tsx | Removes QuickActions usage from media group layout |
| app/(media)/about/page.tsx | Comments out sections/imports and adjusts spacing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <section className="pb-12 px-4 lg:px-8" id="github"> | ||
| <div className="max-w-6xl mx-auto"> | ||
| <GitHubShowcase /> | ||
| </div> | ||
| </section> |
There was a problem hiding this comment.
This introduces a duplicate id=\"github\" in the DOM because GitHubShowcase itself renders a <section id=\"github\">.... Duplicate IDs can break anchor navigation and accessibility. Remove the wrapper id, or update GitHubShowcase to accept an optional id prop so only one element owns the anchor.
| import GitHubShowcase from "@/components/github/GitHubShowcase"; | ||
| import { ProjectsShowcase } from "@/components/projects"; | ||
|
|
||
| export default function LibraryPage() { |
There was a problem hiding this comment.
The component name LibraryPage doesn’t match the route (/projects). Renaming it to something like ProjectsPage avoids confusion when debugging stack traces and React DevTools.
| export default function LibraryPage() { | |
| export default function ProjectsPage() { |
| variant={selectedCategory === category ? "default" : "outline"} | ||
| size="sm" | ||
| onClick={() => onCategoryChange(category)} | ||
| className="transition-all duration-200 cursor-pointer hover:bg-primary/10" |
There was a problem hiding this comment.
These buttons act like a single-select toggle group, but the selected state isn’t exposed to assistive tech. Add an accessibility attribute such as aria-pressed={selectedCategory === category} (or consider a tablist pattern with role=\"tablist\"/role=\"tab\" + aria-selected) so screen readers can understand which filter is active.
| className="transition-all duration-200 cursor-pointer hover:bg-primary/10" | |
| className="transition-all duration-200 cursor-pointer hover:bg-primary/10" | |
| aria-pressed={selectedCategory === category} |
| href="/media" | ||
| href="/blog" | ||
| className="flex items-center gap-1.5" | ||
| aria-label="Go to home page" |
There was a problem hiding this comment.
The aria-label says “Go to home page” but the link now points to /blog. Update the label to match the destination (e.g., “Go to blogs” / “Go to blog index”) to avoid misleading screen reader users.
| aria-label="Go to home page" | |
| aria-label="Go to blog index" |
| import { Card } from "@/components/ui/card"; | ||
| //import { Card, CardContent } from "@/components/ui/card"; |
There was a problem hiding this comment.
There are several commented-out imports and large commented blocks left in the page. This makes the file harder to maintain and can hide accidental regressions. Prefer removing dead code entirely (or gating it behind a feature flag/config) rather than keeping commented code in the main branch.
| import type { AboutTranslations } from "@/types/configs/i18n"; | ||
| import Image from "next/image"; | ||
| import { motion } from "framer-motion"; | ||
| //import { motion } from "framer-motion"; |
There was a problem hiding this comment.
There are several commented-out imports and large commented blocks left in the page. This makes the file harder to maintain and can hide accidental regressions. Prefer removing dead code entirely (or gating it behind a feature flag/config) rather than keeping commented code in the main branch.
| /* const fadeInUp = { | ||
| hidden: { opacity: 0, y: 20 }, | ||
| visible: { opacity: 1, y: 0 }, | ||
| }; | ||
| }; */ |
There was a problem hiding this comment.
There are several commented-out imports and large commented blocks left in the page. This makes the file harder to maintain and can hide accidental regressions. Prefer removing dead code entirely (or gating it behind a feature flag/config) rather than keeping commented code in the main branch.
No description provided.