Feature/color update #5
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR implements a comprehensive color scheme redesign, replacing the previous multi-color palette (primary, secondary, accent, lightAccent) with a single unified "mainColor" palette. The changes simplify the theme configuration and create a more cohesive visual identity across the application.
Key changes:
- Consolidated four color palettes into one "mainColor" palette with neutral tones
- Replaced Button components with Anchor/Text components for navigation elements
- Removed explicit color props throughout the UI to rely on theme defaults
- Updated navigation structure and removed AppShell.Header wrapper
Reviewed Changes
Copilot reviewed 9 out of 15 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/main.tsx | Replaces four color palettes with single "mainColor" palette, removes Title color defaults, adds Anchor component styling |
| src/App.tsx | Removes AppShell.Header wrapper and theme-based background colors |
| src/components/Navigation.tsx | Replaces Buttons with Text components, updates layout structure, removes color props |
| src/components/ContactLinks.tsx | Comments out color prop from Button components |
| src/pages/Home.tsx | Replaces Buttons with Anchor components, removes color props, updates grid layout |
| src/pages/About.tsx | Updates color references from "primary" to "mainColor" |
| src/pages/Contact.tsx | Removes explicit color props from Text components |
| src/pages/Projects.tsx | Updates all status colors to "mainColor", removes explicit color props |
| README.md | Removes deprecated color scheme documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import { Avatar, Burger, Container, Drawer, Group, Stack, Text, useMantineTheme } from '@mantine/core' | ||
| import { useDisclosure } from '@mantine/hooks' | ||
| import { NavLink, useLocation } from 'react-router-dom' | ||
| import bogdanPhoto from '../assets/bogdan_photo_4.jpeg' |
There was a problem hiding this comment.
The imported bogdanPhoto is not used anywhere in this file. Remove this unused import.
| import bogdanPhoto from '../assets/bogdan_photo_4.jpeg' |
| rel="noreferrer" | ||
| variant={index === 0 ? 'filled' : 'light'} | ||
| color={index === 0 ? 'primary' : 'accent'} | ||
| // color={index === 0 ? 'primary' : 'accent'} |
There was a problem hiding this comment.
Remove this commented-out code rather than leaving it in the codebase. The color prop has been intentionally removed as part of the color scheme refactor.
| // color={index === 0 ? 'primary' : 'accent'} |
No description provided.