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

TreeView item add disabled option: Whether the current item is disabled #1023

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

imxood
Copy link

@imxood imxood commented Feb 11, 2024

Pre-launch Checklist

  • I have updated CHANGELOG.md with my changes
  • I have run "dart format ." on the project
  • I have added/updated relevant documentation

Copy link
Owner

@bdlukaa bdlukaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would love to see what's the UI output of this as well. Thanks.

lib/src/controls/navigation/tree_view.dart Outdated Show resolved Hide resolved
example/lib/screens/navigation/tree_view.dart Outdated Show resolved Hide resolved
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

However, when the parent TreeViewItem is checked, all child TreeViewItems can still be selected.

In my application, if the parent TreeViewItem is disabled, then manually set the disabled property to true for all child elements.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this can be misleading. Maybe adding this to the documentation OR creating a new property (maybe disableChildren) that will disable the children if the parent is disabled?

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