Skip to content

Conversation

@wereHamster
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 21, 2025 13:37
@vercel
Copy link

vercel bot commented Apr 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
timvir ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 21, 2025 1:38pm

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes dark mode styling issues by updating how dark mode selectors are applied across various components. The key changes include replacing the :global dark mode selectors with a direct :root selector, updating a background-color to a CSS variable in the Code theme, and ensuring uniform styling across components.

Reviewed Changes

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

Show a summary per file
File Description
pkg/timvir/search/SearchBoxListItem/SearchBoxListItem.tsx Updated dark mode selector syntax for proper styling
pkg/timvir/search/SearchBoxInput/SearchBoxInput.tsx Simplified dark mode selector syntax
pkg/timvir/search/Search/internal/Dialog.tsx Revised dark mode background selector
pkg/timvir/blocks/WebLink/WebLink.tsx Adjusted dark mode box-shadow styling
pkg/timvir/blocks/Exhibit/Exhibit.tsx Changed dark mode background image reference
pkg/timvir/blocks/Code/theme.ts Updated dark mode background-color to a CSS variable
pkg/timvir/blocks/Code/Code.tsx Modified dark mode styling for code syntax highlighting
.changeset/six-eyes-mix.md Documented the dark mode styling fix in the changeset file

@wereHamster wereHamster merged commit 7d57cda into master Apr 21, 2025
10 checks passed
@wereHamster wereHamster deleted the fix-dark-mode branch April 21, 2025 13:39
@github-actions github-actions bot mentioned this pull request Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants