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.

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

[Bug Report] v-for order is not preserved when used inside an #activator slot. #9632

Closed
ghost opened this issue Nov 8, 2019 · 0 comments
Closed

[Bug Report] v-for order is not preserved when used inside an #activator slot. #9632

ghost opened this issue Nov 8, 2019 · 0 comments
Assignees
@KaelWD
Labels
C: VDialog VDialog C: VMenu VMenu C: VTooltip VTooltip has workaround T: bug Functionality that does not work as intended/expected
Milestone
v3.0.0

Comments

@ghost
Copy link

ghost commented Nov 8, 2019

Environment

Vuetify Version: 2.1.9
Vue Version: 2.6.10
Browsers: Firefox 70.0
OS: Linux x86_64

Steps to reproduce

Use a v-for in an #activator slot.

Expected Behavior

v-for displayed in order (One, Two, Three, Four)

Actual Behavior

The order of the items appears random (One, Four, Three, Two).

Reproduction Link

https://codepen.io/chiln/pen/NWWzXpE?editors=1010

Other comments

John provided a work around: Wrap the v-for in a root element

https://codepen.io/johnjleider/pen/vYYrpMr?editors=1011

Given the workaround, This is not a showstopper, but fixing would avoid confusion & wasted time for new folks.
@ghost ghost added the S: triage label Nov 8, 2019
@KaelWD KaelWD self-assigned this Nov 9, 2019
@KaelWD KaelWD added C: VDialog VDialog C: VMenu VMenu C: VTooltip VTooltip has workaround T: bug Functionality that does not work as intended/expected and removed S: triage labels Nov 9, 2019
jacekkarczmarczyk added a commit that referenced this issue Jan 11, 2020
@jacekkarczmarczyk
fix: correct order of multiple activators
150c159 
fixes #9632
@jacekkarczmarczyk jacekkarczmarczyk mentioned this issue Jan 11, 2020
Closed
9 tasks
@jacekkarczmarczyk jacekkarczmarczyk added the S: has PR The issue has a pending Pull Request label Jan 11, 2020
@jacekkarczmarczyk jacekkarczmarczyk removed the S: has PR The issue has a pending Pull Request label Jan 22, 2020
@KaelWD KaelWD added this to the v3.0.0 milestone May 20, 2021
@KaelWD KaelWD mentioned this issue May 20, 2021
Merged
KaelWD added a commit that referenced this issue May 26, 2021
@KaelWD
feat: create v-dialog and v-overlay components (#13681)
f06f5b4 
Closes #1068
Closes #4807 (class now applies to the root teleported element)
Fixes #6425
Closes #6764
Fixes #7422
Closes #8105
Closes #9595
Fixes #9632
Fixes #11263
@KaelWD KaelWD closed this as completed May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KaelWD KaelWD

Labels
C: VDialog VDialog C: VMenu VMenu C: VTooltip VTooltip has workaround T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging a pull request may close this issue.

fix(detachable): always insert activator after detachable content vuetifyjs/vuetify
2 participants
@jacekkarczmarczyk @KaelWD