[Docs] Fix TooltipProvider missing from Tooltip API Reference section#440
[Docs] Fix TooltipProvider missing from Tooltip API Reference section#440pedromenezes1 wants to merge 4 commits intomainfrom
Conversation
Register TooltipProvider in the component registry as a pass-through sub-component of Tooltip via a new SUB_COMPONENT_OVERRIDES map. No runtime change: TooltipProvider stays as a sibling named export. Adds TooltipBase.Provider props (delay, closeDelay, timeout) to PASSTHROUGH_COMPONENT_DOCS and a TooltipProvider section to the Tooltip docs page.
commit: |
Docs PreviewCommit: |
|
/bonk |
|
I've reviewed all the changes in PR #440. Here's my analysis: Summary of ChangesThis PR fixes the Tooltip component's API Reference section by adding Key Changes:
Review AssessmentThe changes are well-structured and follow the existing codebase patterns. The approach of using an override map is clean and allows documentation for sibling named exports without requiring runtime changes to the component. ✅ Approved The implementation correctly:
|















Fixes the Tooltip component's API Reference section, which was missing
TooltipProviderand its props (delay,closeDelay,timeout).Adds a new
SUB_COMPONENT_OVERRIDESmap to the component-registry codegen so sub-components can be registered for documentation purposes without any runtime change to the component source.TooltipProviderremains a sibling named export.