Skip to content

Fix: scroll-to-top button not clear on dark mode.#345

Merged
Sachinchaurasiya360 merged 1 commit into
Sachinchaurasiya360:mainfrom
shivam5434:patch-1
May 21, 2026
Merged

Fix: scroll-to-top button not clear on dark mode.#345
Sachinchaurasiya360 merged 1 commit into
Sachinchaurasiya360:mainfrom
shivam5434:patch-1

Conversation

@shivam5434
Copy link
Copy Markdown

@shivam5434 shivam5434 commented May 19, 2026

Problem

The scroll-to-top button was invisible on dark mode.

Changes

Section Before After
Dark bg 🔴 invisible ✅ visible (white button)
Light bg ✅ visible ✅ visible

Verification

  • Before:
image
  • After:
image

Type

  • Bug fix

Summary by CodeRabbit

  • Style
    • Added dark mode styling to the back-to-top button, including adjusted background color, text color, and hover state effects for improved visibility in dark theme.

Review Change Stack

Copilot AI review requested due to automatic review settings May 19, 2026 17:07
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 19, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 90deb949-a1b2-4394-b240-8a4312d28f58

📥 Commits

Reviewing files that changed from the base of the PR and between 3552efa and f6e8123.

📒 Files selected for processing (1)
  • client/src/components/common/BackToTopButton.tsx

📝 Walkthrough

Walkthrough

The back-to-top button component receives dark-mode Tailwind utility classes to apply light-themed styling when users are in dark theme: background color, text color, and hover state background color.

Changes

Dark-mode styling for back-to-top button

Layer / File(s) Summary
Add dark-mode styling to BackToTopButton
client/src/components/common/BackToTopButton.tsx
BackToTopButton floating button className now includes dark-mode Tailwind utilities for background, text color, and hover background styling in dark theme.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

bug, level:beginner

Poem

🐰 The button floats in darkness now,
With light-touched text and shadow's bow,
Dark mode hops in to save the day,
When themes flip—buttons stay in play! 🌙✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: fixing visibility of the scroll-to-top button in dark mode by updating styling.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

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 improves the scroll-to-top button’s visibility in dark mode by applying dark-theme-specific background and icon colors.

Changes:

  • Adds dark mode styling to render the button with a white background and black icon.
  • Keeps the existing light mode styling unchanged.

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

@Sachinchaurasiya360 Sachinchaurasiya360 added bug Something isn't working enhancement New feature or request good first issue Good for newcomers level:beginner Good for first-time contributors gssoc:approved Approved for GSSoC scoring labels May 21, 2026
@Sachinchaurasiya360 Sachinchaurasiya360 merged commit 6a38561 into Sachinchaurasiya360:main May 21, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request good first issue Good for newcomers gssoc:approved Approved for GSSoC scoring level:beginner Good for first-time contributors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants