Skip to content

Conversation

@ACTCD
Copy link
Collaborator

@ACTCD ACTCD commented Jul 8, 2025

I tried using light-dark() while doing #828, but I don't know why it didn't work at that time.

Anyway, try checking this again and it works, so replace it.

@ACTCD ACTCD requested a review from Copilot July 8, 2025 21:57
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 replaces the previous JavaScript-based dark-mode color logic for console logs with a static mapping that uses the CSS light-dark() function.

  • Removed the getColor function and media-query listener in favor of a colors constant
  • Updated all console.info calls to import and use the new colors mapping

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/shared/colors.js Swapped dynamic getColor() logic for a static export const colors using light-dark()
src/ext/content-scripts/entry-userscripts.js Updated imports and all console.* calls to reference the new colors entries
Comments suppressed due to low confidence (1)

src/shared/colors.js:5

  • Consider adding a unit test for the colors mapping to verify each key produces the expected CSS string for console styling.
export const colors = {

@ACTCD ACTCD merged commit 77ae5ee into main Jul 8, 2025
1 check passed
@ACTCD ACTCD deleted the refactor-console-colors branch July 8, 2025 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants