Open
Conversation
* feat(navbar): add navigation bar component * Accidentally used dynamic routing not route groups for the components page. Fixed. * Added the Navigation Bar file to the layout.tsx for the aboutpage so it would stick to all pages under the aboutpage. * Changed an import name * feat: added the boston-angel font as a default font under the tailwind-config * feat: Made the text adjust for different device screens * feat: added list for navbar with slight stylings * feat: implemented diagonal animation popup on scroll on card ui * feat: added the AboutTeam template to the main page * feat: added svg assets needed to make the player banner component for the meet the team section. Ongoing component implementation * fix: fixed the font naming as well as layout.tsx stylesheet addition --------- Co-authored-by: OntheBarProg <202311340@fit.edu.ph> Co-authored-by: Seedlign <ricbarrios45@gmail.com> Co-authored-by: mercadoCODING <raphaelandremercado@gmail.com> Co-authored-by: Kourai9 <luigikarlblimos@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This PR merges the revamped About pages and related UI updates, enhances Tailwind configuration, and adds a TRPC router for events.
- Added custom fonts and colors to
tailwind.config.ts - Introduced
eventRouterwithgetEvent,getOrganization, andgetTags, and wired it intoappRouter - Implemented client and server pages for
/aboutand/events/[slug], including GSAP animations, carousel, stars background, and play button
Reviewed Changes
Copilot reviewed 31 out of 41 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| tailwind.config.ts | Standardized quotes/semicolons; added fontFamily.header and customWhite color |
| src/server/api/routers/event.ts | New TRPC router for event, organization, and tags |
| src/server/api/root.ts | Registered eventRouter under appRouter |
| src/components/ui/stars-background.tsx | Canvas-based starfield with twinkling logic |
| src/components/ui/carousel.tsx | Embla-based carousel with keyboard support |
| src/app/about/page.tsx | Main About page with GSAP text and scroll animations |
| src/app/events/[slug]/page.tsx | Event detail page fetching via TRPC and rendering UI |
| src/app/layout.tsx | Removed root layout (no fallback layout remains) |
| src/app/sampleuseoffonts.jsx | Sample font-usage component |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (2)
src/app/sampleuseoffonts.jsx:16
- React component names should be PascalCase. Rename
sampleuseoffontstoSampleUseOfFonts.
const sampleuseoffonts = () => {
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o fix/merge-aboutpage ehem
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.