Skip to content

fix(tooltip): fixed bugged arrow in dark theme - #55

Merged
kampsy merged 1 commit into
kampsy:mainfrom
Aarish8115:tooltip
Aug 18, 2026
Merged

fix(tooltip): fixed bugged arrow in dark theme#55
kampsy merged 1 commit into
kampsy:mainfrom
Aarish8115:tooltip

Conversation

@Aarish8115

@Aarish8115 Aarish8115 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description

Arrowhead of the tooltip component in the dark theme is bugged.

Type of Change

  • Bug fix
  • Refactoring

How Has This Been Tested?

  • Unit tests
  • Integration tests
  • Manual testing

Test Configuration:

  • OS: Windows 11
  • Browser: Brave
  • Node.js version: 24.19.0

Checklist

  • My code follows the project's style guidelines.
  • I have performed a self-review of my code.
  • My changes generate no new warnings or errors.
  • New and existing unit tests pass locally with my changes.

Screenshots (if applicable)

Before:
image

After:
image

Additional Notes

Summary by CodeRabbit

  • Bug Fixes
    • Improved tooltip arrow styling across all positions and visual styles.
    • Ensured arrow colors and transparent borders display correctly when other styles could conflict.

@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown

@Aarish8115 is attempting to deploy a commit to the Kampsy's projects Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3ffe7b3f-a1bc-48e6-a580-1b0dc867f28c

📥 Commits

Reviewing files that changed from the base of the PR and between 0afd80f and ac3e6bc.

📒 Files selected for processing (1)
  • src/lib/tooltip/tooltip.svelte

Included review availability: Your plan includes up to 2 reviews per rolling hour; 1 remains after this review.


📝 Walkthrough

Walkthrough

Changes

Tooltip arrow styles

Layer / File(s) Summary
Apply important arrow utilities
src/lib/tooltip/tooltip.svelte
Tooltip arrow border color and transparency classes use important variants across default, success, error, warning, and violet styles.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to ac3e6

This localized tooltip styling fix changes dark-theme arrow rendering, and no actionable merge-blocking risk remains after normal checks and review.

Possibly related PRs

  • kampsy/ui#44: Both changes update Tailwind tooltip arrow utility classes in src/lib/tooltip/tooltip.svelte.

Suggested reviewers: kampsy

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing a tooltip arrow bug in the dark theme, which aligns with the summary and objectives.
Description check ✅ Passed The description includes all required sections with appropriate content: bug fix type, testing performed with configuration details, checklist completion, and visual evidence (before/after screenshots).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 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.

@vercel

vercel Bot commented Aug 18, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
kampsy-ui Ready Ready Preview Aug 18, 2026 10:47am

@kampsy
kampsy merged commit e3e4a51 into kampsy:main Aug 18, 2026
4 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.8.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants