Conversation
AI Code cleanup and minor tweaks
fix: pnpm lockfile update
…ti's image, and remove the LinkedIn field for the entry.
…pdates across various pages and components.
… and enhancing styling for the Team & Mentorship card.
…timizations, and add a new team image.
…red data for improved search visibility
…oved card layout and WebGL shader styling.
There was a problem hiding this comment.
Pull request overview
This PR updates the site’s UI/branding (glass/neo look + new mini navbar) and expands SEO assets/metadata (robots, sitemap, JSON‑LD, per-route metadata), while also removing an unused dependency.
Changes:
- Added
GlassContainerand refactored multiple pages/components to use it (replacing prior glass/liquid patterns). - Introduced a new
MiniNavbarand swapped it in as the global navigation. - Updated SEO surface area: static
public/sitemap.xml, expandedrobots.ts, richer metadata + JSON‑LD in layouts, and updated manifest/llms.txt.
Reviewed changes
Copilot reviewed 39 out of 47 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| public/team/devaansh.jpeg | Team image asset (PR metadata indicates this file). |
| public/sitemap.xml | Adds a static sitemap file under public/. |
| public/llms.txt | Expands LLM/search-engine context content and org details. |
| pnpm-lock.yaml | Lockfile updated to reflect dependency removal. |
| package.json | Removes vaul dependency. |
| components/ui/web-gl-shader.tsx | Simplifies shader output + adjusts canvas sizing/styling. |
| components/ui/mini-navbar.tsx | Adds new responsive mini navbar with animated mobile menu. |
| components/ui/mini-navbar-demo.tsx | Adds a demo page/component for the mini navbar. |
| components/ui/hero-futuristic.tsx | Refactors hero visuals to use GlassContainer and updates layout/typography. |
| components/ui/glowing-card.tsx | Updates card visuals/hover effects to match new glass styling. |
| components/ui/globe.tsx | Performance adjustments + changes initial camera/rotation logic. |
| components/ui/glass-container.tsx | Introduces reusable glass container component with optional glow/animation. |
| components/ui/flickering-footer.tsx | Updates footer copy. |
| components/ui/features-8.tsx | Refactors feature cards to use GlassContainer. |
| components/ui/entropy.tsx | Adds visibility-based pausing + performance throttling for canvas animation. |
| components/ui/3d-card.tsx | Throttles mousemove via requestAnimationFrame and tweaks transitions/types. |
| components/team-globe.tsx | Adjusts globe rotate speed on mobile + adds new arc. |
| components/team-case-study.tsx | Optimizes color sampling + refactors card layout/typography. |
| components/navigation.tsx | Replaces existing navigation with MiniNavbar. |
| components/infinite-moving-cards-demo.tsx | Fixes team image path for Devansh. |
| app/sitemap.ts | Removes Next.js metadata sitemap route implementation. |
| app/robots.ts | Expands crawler rules, adds host, and points sitemap to public/sitemap.xml. |
| app/projects/layout.tsx | Improves metadata (keywords, canonical, OG). |
| app/page.tsx | Refactors homepage sections to use GlassContainer and adds shader loading. |
| app/manifest.ts | Expands PWA manifest fields and adds shortcuts. |
| app/layout.tsx | Adds viewport config, richer global metadata, and multiple JSON‑LD blocks. |
| app/join/page.tsx | Refactors join page UI to use GlassContainer. |
| app/join/layout.tsx | Adds route-specific metadata for /join. |
| app/impact/page.tsx | Refactors impact page UI to use GlassContainer. |
| app/impact/layout.tsx | Adds route-specific metadata for /impact. |
| app/globals.css | Tunes shadows/glows and adds rendering/backdrop-filter performance tweaks. |
| app/faq/page.tsx | Refactors FAQ page UI to use GlassContainer. |
| app/faq/layout.tsx | Adds route metadata + FAQ JSON‑LD. |
| app/events/layout.tsx | Adds route metadata + events JSON‑LD. |
| app/contact/page.tsx | Refactors contact page UI to use GlassContainer. |
| app/contact/layout.tsx | Adds route-specific metadata for /contact. |
| app/coc/page.tsx | Refactors CoC page UI to use GlassContainer. |
| app/coc/layout.tsx | Adds route metadata for /coc. |
| app/about/page.tsx | Updates team/volunteer entries and images. |
| app/about/layout.tsx | Adds route-specific metadata for /about. |
| .gitignore | Ignores .env* recursively and whitelists .env.example. |
| .env.example | Adds example env vars for API keys. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| // Focus Areas are now handled within the Features component | ||
|
|
||
| import { GlassContainer } from "@/components/ui/glass-container"; |
There was a problem hiding this comment.
import { GlassContainer ... } appears after other statements (the stats constant). ES modules require all imports to be at the top level before other code; this will fail to compile. Move this import up with the other imports (and keep all imports grouped before any const/dynamic() declarations).
| import { GlassContainer } from "@/components/ui/glass-container" | ||
|
|
There was a problem hiding this comment.
import { GlassContainer ... } is declared after the fieldClass constant. ES module imports must be at the top of the file; this will fail to compile. Move this import up with the other imports.
| @@ -1,15 +1,96 @@ | |||
| import { Metadata } from "next"; | |||
| import Script from "next/script"; | |||
There was a problem hiding this comment.
Script is imported from next/script but never used (the component renders a lowercase <script>). This will trigger unused-import linting in many setups. Remove the unused import, or switch to <Script> and use the imported component.
| import Script from "next/script"; |
| import { Shield, Users, Rocket, Lightbulb, CodeXml } from 'lucide-react' | ||
| import Image from 'next/image' | ||
| import { GlassContainer } from '@/components/ui/glass-container' | ||
| import { cn } from '@/lib/utils' |
There was a problem hiding this comment.
cn is imported here but not used anywhere in this file. Please remove the unused import to avoid lint/build failures in setups that enforce unused imports.
| import { cn } from '@/lib/utils' |
| @@ -10,13 +12,6 @@ import { | |||
| } from "lucide-react"; | |||
There was a problem hiding this comment.
There are several unused imports at the top of this file (Link, multiple Lucide icons, Card*, Button). If linting/type-checking for unused imports is enabled, this will fail CI/build. Please remove unused imports or reintroduce usage as intended.
| <> | ||
| <WebGLShader /> | ||
| <div className="flex flex-col w-full max-w-full overflow-x-hidden"> |
There was a problem hiding this comment.
The page renders <WebGLShader /> here, but HeroFuturistic already renders its own WebGLShader background. This will mount two canvases/shaders on the homepage and can cause visual layering + unnecessary GPU load. Remove one of them (prefer keeping the shader inside HeroFuturistic, or pass a flag so it only renders once).
| }, | ||
| ]; | ||
|
|
||
| import { GlassContainer } from "@/components/ui/glass-container"; | ||
|
|
||
| export default function Join() { |
There was a problem hiding this comment.
This file now has import statements after non-import code (import Script ... after the WebGLShader const and import { GlassContainer ... } after the faqs constant). ES modules require imports to come before other statements; this will fail to compile. Please move all imports to the top of the file (above the dynamic() calls and constants).
| import { GlassContainer } from "@/components/ui/glass-container"; | ||
|
|
There was a problem hiding this comment.
import { GlassContainer ... } is declared after the faqs array. ES module imports must be at the top of the file; this will fail to compile. Move this import up with the other imports.
| 'use client' | ||
| import { useEffect, useRef } from 'react' | ||
| import React, { useEffect, useRef, useState } from 'react' | ||
|
|
There was a problem hiding this comment.
useState is imported but not used (state is created via React.useState). This is an unused import and will fail linting in many configurations. Either remove useState from the import list or switch to useState(...) instead of React.useState(...).
| const phi = (90 - lat) * (Math.PI / 180); | ||
| const theta = (lng + 280) * (Math.PI / 180); | ||
| // 280 degree hardcode to spawn globe directly at lucknow | ||
|
|
There was a problem hiding this comment.
theta adds a hardcoded + 280 offset, which makes InitialRotation not actually respect the provided lat/lng for general use (it will be wrong for any non-Lucknow initialPosition). If the goal is “face (lat,lng)”, compute the offset from globe orientation once (or document/encode it as a constant with a clear name) and apply it consistently rather than hardcoding a Lucknow-specific value inside this generic helper.
…ite interaction, team queries, and contact form submission, along with its glowing UI component.
…ro, glowing AI chat assistant UI, and assistant API endpoint.
…, and add various UI components including TeamCaseStudy, HeroFuturistic, Features, GlassContainer, FlickeringFooter, and MiniNavbar.
…, and `MiniNavbar` along with supporting image assets.
…uristic UI components.
Co-authored-by: a3ro-dev <98264660+a3ro-dev@users.noreply.github.com>
Co-authored-by: a3ro-dev <98264660+a3ro-dev@users.noreply.github.com>
…EO, and dynamic background components.
…ce optimizations, hardware detection, and static fallback.
…and graceful degradation
…ance optimizations, hardware detection, and a static fallback.
…toring and inject build-time git information into environment variables.
… performance optimization and hardware-based fallbacks.
…-in performance monitoring and reduced motion fallback.
…eature detection, and real-time performance diagnostics.
…shader background.
…nd add new static content pages.
…s.md and llms.txt Co-authored-by: a3ro-dev <98264660+a3ro-dev@users.noreply.github.com>
…nd-form Fix contact email, improve mobile responsiveness, and update AI context files
Remove hardcoded "300" from top teams copy and fix event date to single day
…d supporting UI components.
…rsistence, and a new prompt input component.
… assistant component.
…I, Q&A chat interface, and update agent documentation.
… and impact pages.
…2026, merging Cybersecurity into Open Innovation and revising prize pool information.
…dget integration and emphasizing Discord link preference over WhatsApp for community engagement.
No description provided.