docs(button): remove reference to non-existent iconOnly prop#1680
docs(button): remove reference to non-existent iconOnly prop#1680farhan523 wants to merge 1 commit into
Conversation
The Button docs referenced an iconOnly prop that was removed in flowbite-react@0.11.0 and no longer exists on the component. Reword the section to describe the actual usage shown in the example (icon passed as the button's only child). Fixes themesberg#1589
|
@farhan523 is attempting to deploy a commit to the Bergside Team on Vercel. A member of the Team first needs to authorize it. |
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated documentation for the Button component's "icon-only" example, replacing the reference to a nonexistent ChangesButton documentation update
Estimated code review effort: 1 (Trivial) | ~2 minutes Related issues: Fixes reference to a nonexistent Suggested labels: documentation Suggested reviewers: None Poem A rabbit hopped through docs so neat, 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration. 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. Comment |
Summary
The "Button with only icons" section of the Button docs tells users to add an
iconOnlyprop to<Button>, but that prop no longer exists — it was removed inflowbite-react@0.11.0. The code example in that section already shows the correct usage (passing the icon as the button's only child), so only the descriptive text was wrong.This PR rewords the sentence to describe the actual usage, keeping the section and example intact as suggested by the maintainers in the issue discussion.
Related issues
Fixes #1589
No API or theme changes — documentation only.
Summary by CodeRabbit