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 #6371: Add node and expanded data on nodetoggleicon. #6372

Conversation

mehdirande
Copy link

Nodetoggleicon slot is used to customize expand / collapse and loading icon, but when loading mode is set to icon mode and a node is loading, TreeNode use nodetoggleicon without any props instead of binding node and expanded.
This commit bind node and expanded props to nodetoggleicon slot when loading mode is set to icon and node.loading is true.

Copy link

vercel bot commented Sep 10, 2024

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

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
primevue ⬜️ Ignored (Inspect) Visit Preview Sep 10, 2024 0:28am
primevue-v3 ⬜️ Ignored (Inspect) Visit Preview Sep 10, 2024 0:28am

Nodetoggleicon slot is used to customize expand / collapse and loading
icon, but when loading mode is set to icon mode and a node is loading,
TreeNode use nodetoggleicon without any props instead of binding node and
expanded.
This commit bind node and expanded props to nodetoggleicon slot when
loading mode is set to icon and node.loading is true.
@mehdirande mehdirande force-pushed the fix-6371-add-slots-data-on-tree-nodetoggleicon branch from 31e2ddf to 264b30c Compare September 10, 2024 12:28
@mehdirande mehdirande changed the title Fix #6371: Add node and expanded date on nodetoggleicon. Fix #6371: Add node and expanded data on nodetoggleicon. Sep 10, 2024
@tugcekucukoglu
Copy link
Member

Please create an issue while sending PR.

@tugcekucukoglu tugcekucukoglu added the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label Sep 10, 2024
@mehdirande
Copy link
Author

Please create an issue while sending PR.

I created ticket #6371, which is mentioned in the commit message, beforehand. Is this not the correct way to do it? If not, could you please let me know what I did wrong so I can improve for next time?

@tugcekucukoglu tugcekucukoglu added this to the 4.1.0 milestone Sep 11, 2024
@tugcekucukoglu tugcekucukoglu removed the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label Sep 11, 2024
@tugcekucukoglu tugcekucukoglu removed this from the 4.1.0 milestone Sep 24, 2024
@tugcekucukoglu tugcekucukoglu merged commit e4d431e into primefaces:master Sep 24, 2024
3 of 5 checks passed
@mehdirande mehdirande deleted the fix-6371-add-slots-data-on-tree-nodetoggleicon branch September 24, 2024 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants