Skip to content

[Bug Report][3.7.9] VTreeView update:activated will be fired if a non activateable node is activated #20946

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.

Sign up for GitHub

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

Jump to bottom
Closed
mueller-jens opened this issue Jan 30, 2025 · 1 comment · Fixed by #20951
Closed

[Bug Report][3.7.9] VTreeView update:activated will be fired if a non activateable node is activated #20946

mueller-jens opened this issue Jan 30, 2025 · 1 comment · Fixed by #20951
Assignees
@yuwu9145
Labels
C: VTreeview P: low The issue is of low importance T: bug Functionality that does not work as intended/expected
Milestone
v3.7.x

Comments

@mueller-jens
Copy link
Contributor

mueller-jens commented Jan 30, 2025 •
edited
Loading

Environment

Vuetify Version: 3.7.9
Vue Version: 3.4.27
Browsers: Edge 132.0.0.0
OS: Windows 10

Steps to reproduce

  1. Open the link and the dev console of the browser
  2. click on a leave and you see that update:activated is fired with the id of the leaf
  3. click on a node (at the name) with children (for instance "public") you see that update:activated is fired with the id last activated leaf

Expected Behavior

in step 3 update:activated will not be fired, because there is no change

Actual Behavior

In step 3 update:activated is fired with the id last activated leaf

Reproduction Link

https://play.vuetifyjs.com/#...
@mueller-jens
Copy link
Contributor Author

It looks like this only happens, if open-on-click is not set

@yuwu9145 yuwu9145 self-assigned this Jan 31, 2025
@yuwu9145 yuwu9145 added T: bug Functionality that does not work as intended/expected C: VTreeview and removed S: triage labels Jan 31, 2025
@yuwu9145 yuwu9145 mentioned this issue Jan 31, 2025
Merged
@yuwu9145 yuwu9145 added the P: low The issue is of low importance label Feb 1, 2025
@KaelWD KaelWD added this to the v3.7.x milestone Feb 11, 2025
@KaelWD KaelWD closed this as completed in ea70e18 Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuwu9145 yuwu9145

Labels
C: VTreeview P: low The issue is of low importance T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
v3.7.x
Development

Successfully merging a pull request may close this issue.

fix(nested): prevent parent node activate firing for singleLeaf vuetifyjs/vuetify
3 participants
@yuwu9145 @mueller-jens @KaelWD