Skip to content
This repository was archived by the owner on Jul 3, 2026. It is now read-only.

Merge pull request #55 from frlund3/claude/review-project-164jY - #56

Merged
frlund3 merged 2 commits into
mainfrom
dev
Feb 2, 2026
Merged

Merge pull request #55 from frlund3/claude/review-project-164jY#56
frlund3 merged 2 commits into
mainfrom
dev

Conversation

@frlund3

@frlund3 frlund3 commented Feb 2, 2026

Copy link
Copy Markdown
Owner

No description provided.

claude and others added 2 commits February 2, 2026 23:10
- Tema-toggle-knapp lagt til i mobil-headeren på forsiden
- Hardkodet text-white/text-gray-300/border-gray-600 i FilterSheet erstattet
  med text-theme/text-theme-secondary/border-theme for lyst tema-støtte
- Overskrifter i filter-sheet bruker nå text-theme

https://claude.ai/code/session_01751H62CcNYsAXtyrLWzwnm
Copilot AI review requested due to automatic review settings February 2, 2026 23:11
@vercel

vercel Bot commented Feb 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
brannloggen Ready Ready Preview, Comment Mar 23, 2026 1:26pm
brannloggen-nha2 Ready Ready Preview, Comment Mar 23, 2026 1:26pm

Request Review

@frlund3
frlund3 merged commit cb18fab into main Feb 2, 2026
1 of 3 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request standardizes the theme system across UI components by replacing hardcoded color classes with theme-based CSS variables. The changes ensure consistent styling that adapts to the application's theme.

Changes:

  • Replaced hardcoded color classes (border-gray-600, text-gray-300, text-white) with theme variables (border-theme, text-theme-secondary, text-theme)
  • Added a theme toggle button to the mobile header with appropriate sun/moon icons

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/public/FilterSheet.tsx Updated checkbox borders, text colors, and headings to use theme CSS variables instead of hardcoded gray/white colors
src/app/page.tsx Added theme toggle button to mobile header with conditional icon rendering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants