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

Accept array as PT value #5647

Merged
merged 3 commits into from
Dec 27, 2023
Merged

Accept array as PT value #5647

merged 3 commits into from
Dec 27, 2023

Conversation

kyjus25
Copy link
Contributor

@kyjus25 kyjus25 commented Dec 26, 2023

Defect Fixes

PrimeReact required the use of an internal classNames utility for array class joining. Hoist this out so it matches the Vue implementation. Remains backwards compatible.

https://github.com/primefaces/primereact/blob/master/components/lib/passthrough/tailwind/index.js#L613
https://github.com/primefaces/primevue/blob/master/components/lib/passthrough/tailwind/index.js#L741

Feature Requests

Due to company policy, we are unable to accept feature request PRs with significant changes as such cases has to be implemented by our team following our own processes.
Smaller scaled feature implementations such as adding a property to a component will be considered for merging.

Copy link

vercel bot commented Dec 26, 2023

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 Dec 27, 2023 4:14am
primereact-v9 ⬜️ Ignored (Inspect) Visit Preview Dec 27, 2023 4:14am

Copy link

Thanks a lot for your contribution! But, PR does not seem to be linked to any issues. Please manually link to an issue or mention it in the description using #<issue_id>.

@melloware melloware added this to the 10.2.2 milestone Dec 26, 2023
@melloware melloware added the Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add label Dec 26, 2023
Copy link

Thanks a lot for your contribution! But, PR does not seem to be linked to any issues. Please manually link to an issue or mention it in the description using #<issue_id>.

Copy link

Thanks a lot for your contribution! But, PR does not seem to be linked to any issues. Please manually link to an issue or mention it in the description using #<issue_id>.

@melloware melloware merged commit 849287e into primefaces:master Dec 27, 2023
2 of 5 checks passed
@melloware
Copy link
Member

i prefer it match the Vue implementation I did not realize Vue supported that out of the box so this is a good change if React does not support it natively to keep them in sync.

@melloware melloware mentioned this pull request Jan 9, 2024
88 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants