Skip to content

Conversation

@weebao
Copy link
Member

@weebao weebao commented Jun 9, 2025

No description provided.

Copilot AI review requested due to automatic review settings June 9, 2025 08:03
@weebao weebao merged commit ca33809 into main Jun 9, 2025
1 check passed
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 pull request adjusts the height configuration for the dithering canvas effect.

  • Updated the height calculation from 4rem to 5rem for better visual alignment.

<div class="fixed -left-4 top-0 -z-[20] opacity-20">
<!-- Cool dithering Japanese effect -->
<div data-us-project="jzsvsRi2fATTRNcOAqWq" class="h-[calc(100dvh+4rem)] w-screen"></div>
<div data-us-project="jzsvsRi2fATTRNcOAqWq" class="h-[calc(100dvh+5rem)] w-screen"></div>
Copy link

Copilot AI Jun 9, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider adding a comment or extracting the '5rem' value to a constant to clarify the rationale behind this adjustment and facilitate future maintenance.

Suggested change
<div data-us-project="jzsvsRi2fATTRNcOAqWq" class="h-[calc(100dvh+5rem)] w-screen"></div>
<div data-us-project="jzsvsRi2fATTRNcOAqWq" class={`h-[calc(100dvh+${EXTRA_HEIGHT})] w-screen`}></div>

Copilot uses AI. Check for mistakes.
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.

1 participant