Skip to content
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.

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

Fix #5983: MergeProps refactor and optimize #5996

Merged
merged 1 commit into from
Feb 18, 2024

Conversation

melloware
Copy link
Member

Fix #5983: MergeProps refactor and optimize

  • Optimize mergeProps method to short circuit and not call isFunction as many times
  • Removed _mergeProps for one cleaned up mergeProps
  • TS Doc of mergeProps
  • All components now using useMergeProps
  • TreeTableRow now matches DT BodyRow functionality

cc @Et7f3

@melloware melloware added the Core Team Issue or pull request has been *opened* by a member of Core Team label Feb 18, 2024
Copy link

vercel bot commented Feb 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Comments Updated (UTC)
primereact ⬜️ Ignored (Inspect) Visit Preview Feb 18, 2024 3:05pm
primereact-v9 ⬜️ Ignored (Inspect) Visit Preview Feb 18, 2024 3:05pm

@melloware melloware merged commit 511658c into primefaces:master Feb 18, 2024
7 checks passed
@melloware melloware deleted the PR5983 branch February 18, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Team Issue or pull request has been *opened* by a member of Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DataTable: rowClassName don't take prevalence over passthrough tailwind
1 participant