Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix tooltip code element wrapping #3149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
steven-tey merged 1 commit into from
Nov 24, 2025
Merged

Fix tooltip code element wrapping #3149

steven-tey merged 1 commit into from
Nov 24, 2025

Conversation

@TWilson023
Copy link
Collaborator

@TWilson023 TWilson023 commented Nov 24, 2025 •
edited by coderabbitai bot
Loading

Before After
Screenshot 2025-11-24 at 10 28 52 AM Screenshot 2025-11-24 at 10 24 36 AM

Summary by CodeRabbit

  • Style
    • Improved styling of code blocks displayed within tooltips for enhanced readability and visual clarity.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link
Contributor

vercel bot commented Nov 24, 2025 •
edited
Loading

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

Project Deployment Preview Updated (UTC)
dub Ready Ready Preview Nov 24, 2025 3:33pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 24, 2025 •
edited
Loading

Walkthrough

The change adds two Tailwind CSS classes (prose-code:inline-block and prose-code:leading-none) to the Markdown rendering container in the tooltip component to adjust the styling of inline code elements within tooltip content.

Changes

Cohort / File(s) Summary
Tooltip styling
packages/ui/src/tooltip.tsx		 Added prose-code:inline-block and prose-code:leading-none Tailwind classes to control code block rendering in TooltipMarkdown

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • #3144 — Also modifies packages/ui/src/tooltip.tsx to adjust CSS classes applied to TooltipMarkdown styling
  • #3083 — Tweaks Tailwind classes within TooltipMarkdown in the same tooltip component file

Suggested reviewers

  • steven-tey

Poem

🐰 A tooltip adorned with classes so neat,
inline-block makes code display complete,
With leading-none spacing just right,
Our rendered prose shines ever so bright!
✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix tooltip code element wrapping' clearly describes the main change: adding Tailwind classes to fix code element wrapping in tooltip Markdown rendering.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tooltip-code-fix
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 070649f and c041980.

📒 Files selected for processing (1)
  • packages/ui/src/tooltip.tsx (1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings 
Learnt from: TWilson023
Repo: dubinc/dub PR: 2872
File: apps/web/app/(ee)/partners.dub.co/(dashboard)/profile/profile-details-form.tsx:180-189
Timestamp: 2025-09-24T15:50:16.414Z
Learning: TWilson023 prefers to keep security vulnerability fixes separate from refactoring PRs when the vulnerable code is existing and was only moved/relocated rather than newly introduced.
📚 Learning: 2025-09-24T16:09:52.724Z 
Learnt from: TWilson023
Repo: dubinc/dub PR: 2872
File: apps/web/ui/partners/online-presence-form.tsx:181-186
Timestamp: 2025-09-24T16:09:52.724Z
Learning: The cn utility function in this codebase uses tailwind-merge, which automatically resolves conflicting Tailwind classes by giving precedence to later classes in the className string. Therefore, patterns like `cn("gap-6", variant === "settings" && "gap-4")` are valid and will correctly apply gap-4 when the condition is true.

Applied to files:

  • packages/ui/src/tooltip.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (1)
packages/ui/src/tooltip.tsx (1)

33-33: LGTM! Targeted fix for code element layout.

The added classes improve inline code rendering in tooltips: inline-block provides better wrapping control, and leading-none eliminates excess vertical space that can misalign code with surrounding text.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.

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.

@steven-tey steven-tey merged commit ad5445c into main Nov 24, 2025
8 of 9 checks passed
@steven-tey steven-tey deleted the tooltip-code-fix branch November 24, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@steven-tey steven-tey steven-tey approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TWilson023 @steven-tey