feat(FloatButtonGroup): Add onClick prop

[Riyad-Arafat]

[中文版模板 / Chinese template]

🤔 This is a ...

• New feature
• Bug fix
• Site / documentation update
• Demo update
• Component style update
• TypeScript definition update
• Bundle size optimization
• Performance optimization
• Enhancement feature
• Internationalization
• Refactoring
• Code style optimization
• Test Case
• Branch merge
• Workflow
• Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language	Changelog
🇺🇸 English	fix: missing onClick in FloatButtonGroup
🇨🇳 Chinese	修复: FloatButtonGroup 遗漏 onClick

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

• Doc is updated/provided or not needed
• Demo is updated/provided or not needed
• TypeScript definition is updated/provided or not needed
• Changelog is provided or not needed

[github-actions]

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (dee3901) to head (87135be).

❗ Current head 87135be differs from pull request most recent head b472f4a. Consider uploading reports for the commit b472f4a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           feature    #40706    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          739       708    -31     
  Lines        12688     12101   -587     
  Branches      3334      3207   -127     
==========================================
- Hits         12688     12101   -587     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MadCcc]

Any senario in real world needs this prop? Is onOpenChange enough?

[Riyad-Arafat]

@MadCcc
The reason I added this prop is to enable the ability to update the open prop when clicking on the button. In some cases, this behavior may be required, and the onClick prop provides a convenient way to handle it.

[MadCcc]

@MadCcc The reason I added this prop is to enable the ability to update the open prop when clicking on the button. In some cases, this behavior may be required, and the onClick prop provides a convenient way to handle it.

In this case onOpenChange and trigger="click" is enough.

[Riyad-Arafat]

@MadCcc The reason I added this prop is to enable the ability to update the open prop when clicking on the button. In some cases, this behavior may be required, and the onClick prop provides a convenient way to handle it.

In this case onOpenChange and trigger="click" is enough.

You're correct, but FloatButtonGroupProps extends FloatButtonProps. I became confused when I attempted to use it and it failed to work.

[Riyad-Arafat]

@MadCcc
In my case, I want the menu to remain open even when I click away from the button. I want the open state to be changed only when the float button is clicked.

[li-jia-nan]

We worry it is easy to cause ambiguity, some people will think that the click of the FloatButtonGroup is the entire area, and some people will think it is the trigger button.

[Riyad-Arafat]

We worry it is easy to cause ambiguity, some people will think that the click of the FloatButtonGroup is the entire area, and some people will think it is the trigger button.

Thank you for bringing up the concern about ambiguity. I agree that choosing a clear and meaningful name for the click area would be the best solution to avoid any confusion.
Do you have any suggestions for a more descriptive name that we could use instead?

[AbdullahWins]

good job!

[afc163]

Conflicting

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 87135be:

Sandbox	Source
antd reproduction template (forked)	Configuration

[Riyad-Arafat]

@afc163
resolved

[github-actions]

👁 Visual Regression Report for PR #40706 Passed ✅

🎯 Target branch: feature (dee3901)
📖 View Full Report ↗︎

🎊 Congrats! No visual-regression diff found.