feat: update and unify design of landing, about and further subpages#18
Conversation
github-actions
Bot
added
client-application
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
There was a problem hiding this comment.
Pull request overview
This PR modernizes the landing page, about page, and related subpages with a unified design language aligned with TUM chair messaging. It removes external image dependencies, extracts a shared footer component, and refines the navbar with centered navigation pills and scroll-based styling.
Key Changes
- Unified visual design with glassmorphic effects and consistent color palette across landing and about pages
- Extracted shared footer component to eliminate duplication across multiple page files
- Enhanced navbar with centered navigation pills, active state indicator, and scroll-responsive shadow effects
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| next.config.ts | Simplified configuration by removing empty comment block |
| components/navbar.tsx | Complete navbar redesign with centered pill navigation, scroll detection, and glassmorphic styling |
| components/footer.tsx | New shared footer component extracted from duplicated code |
| app/session/page.tsx | Removed inline footer and "Skipped" stat display |
| app/page.tsx | Major landing page restructure with new hero section, problem statement section, and updated challenges layout |
| app/onboarding/page.tsx | Removed duplicate inline footer markup |
| app/layout.tsx | Added global background gradient and footer component to layout |
| app/dashboard/page.tsx | Removed duplicate inline footer markup |
| app/about/page.tsx | Complete redesign with new hero section, gallery grid, and simplified team/course sections |
Comments suppressed due to low confidence (3)
app/session/page.tsx:130
- Corrected spelling of 'w/full' to 'w-full' in the className.
<header className="relative z-10 mx-auto flex w/full max-w-6xl items-center justify-between px-6 py-10 lg:px-0">
app/about/page.tsx:1
- The 'aria-hidden' attribute requires a boolean value. It should be 'aria-hidden="true"' or 'aria-hidden={true}', not just 'aria-hidden'.
import {
app/about/page.tsx:1
- The 'aria-hidden' attribute requires a boolean value. It should be 'aria-hidden="true"' or 'aria-hidden={true}', not just 'aria-hidden'.
import {
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
|
@MaximilianAnzinger Please merge this PR (#18) and #16 before #21 |
|
Content update for the About page outstanding, once that’s done, ready to merge |
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues
|
Done |
Good to go 🎅 |
🔍 Code Quality Report📊 Code Statistics🚨 Potential Issues✅ No console statements found |
Description
Streamlined landing/About design to align with TUM chair messaging, updated Problem/Challenges layout, and centered navbar pill.
Replaced all Unsplash images with placeholders and removed remote image config.
Type of Change
expected)
Related Issues
Changes Made
next.config.ts.Testing
Test Cases
Manual Testing
Screenshots/Videos