Skip to content

Fix docs info of DefaultIsShown parameter of BitTooltip (#12307)#12308

Merged
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12307-tooltip-defaultisshown-issues
May 4, 2026
Merged

Fix docs info of DefaultIsShown parameter of BitTooltip (#12307)#12308
msynk merged 1 commit into
bitfoundation:developfrom
zahra-yousefi:12307-tooltip-defaultisshown-issues

Conversation

@zahra-yousefi
Copy link
Copy Markdown
Member

@zahra-yousefi zahra-yousefi commented May 4, 2026

closes #12307

Summary by CodeRabbit

  • Bug Fixes
    • Updated Tooltip component initialization behavior by modifying the default parameter handling to support null states, enabling more robust control over component display states and improving initialization consistency.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8430676b-8a50-4959-a48c-577261d98965

📥 Commits

Reviewing files that changed from the base of the PR and between a28c738 and ab12142.

📒 Files selected for processing (1)
  • src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Surfaces/Tooltip/BitTooltipDemo.razor.cs

Walkthrough

This PR corrects metadata documentation for the DefaultIsShown parameter of the BitTooltip component by changing its type from bool to bool? and its default value from false to null to accurately reflect the actual parameter definition.

Changes

Tooltip Parameter Metadata Correction

Layer / File(s) Summary
Metadata Fix
src/BlazorUI/Demo/Client/Bit.BlazorUI.Demo.Client.Core/Pages/Components/Surfaces/Tooltip/BitTooltipDemo.razor.cs
DefaultIsShown parameter type updated from "bool" to "bool?" and default value from "false" to "null" to match actual implementation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tooltip's tale, once told awry,
Now null instead of false—oh my!
The metadata shines, precise and true,
With nullable bools, all fixed for you! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: fixing documentation information for the DefaultIsShown parameter of BitTooltip, with an issue reference.
Linked Issues check ✅ Passed The PR changes the DefaultIsShown parameter metadata from bool to nullable bool? with null default value, directly addressing issue #12307's requirement to fix incorrect Type and DefaultValue documentation properties.
Out of Scope Changes check ✅ Passed All changes are narrowly focused on correcting the DefaultIsShown parameter metadata in BitTooltipDemo.razor.cs, with no out-of-scope modifications present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

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

@msynk msynk merged commit 7abe14f into bitfoundation:develop May 4, 2026
3 checks passed
@zahra-yousefi zahra-yousefi deleted the 12307-tooltip-defaultisshown-issues branch May 4, 2026 19:56
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.

Incorrect info in the docs of the DefaultIsShown parameter of the BitTooltip

2 participants