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

add context menu #398

Merged
merged 8 commits into from
Jun 29, 2022
Merged

add context menu #398

merged 8 commits into from
Jun 29, 2022

Conversation

nidetuzi
Copy link
Contributor

@nidetuzi nidetuzi commented Jun 18, 2022

Add context menu

Add Tree View Secondary Item Event

I don't think I made the my menu well enough

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.

onRightClick -> onSecondaryTap

lib/src/controls/navigation/tree_view.dart Outdated Show resolved Hide resolved
lib/src/controls/navigation/tree_view.dart Outdated Show resolved Hide resolved
lib/src/controls/navigation/tree_view.dart Outdated Show resolved Hide resolved
lib/src/utils/popup.dart Show resolved Hide resolved
@nidetuzi
Copy link
Contributor Author

I can't write comment, so I added a demo to it

@nidetuzi nidetuzi requested a review from bdlukaa June 21, 2022 03:33
@bdlukaa
Copy link
Owner

bdlukaa commented Jun 21, 2022

conflicting files

@bdlukaa bdlukaa merged commit 292fc47 into bdlukaa:master Jun 29, 2022
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.

None yet

2 participants