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-menu no longer rendering all the v-lists #10522

Closed
saravankumarpa opened this issue Feb 8, 2020 · 5 comments · Fixed by #10525 or anyulled/reactivewebflux#5 · May be fixed by SigmaITC/competencebyjava-front-end#8
Closed

[Bug Report] v-menu no longer rendering all the v-lists #10522

saravankumarpa opened this issue Feb 8, 2020 · 5 comments · Fixed by #10525 or anyulled/reactivewebflux#5 · May be fixed by SigmaITC/competencebyjava-front-end#8
Assignees
@jacekkarczmarczyk
Labels
C: VMenu VMenu S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Milestone
v2.2.x

Comments

@saravankumarpa
Copy link

saravankumarpa commented Feb 8, 2020 •
edited
Loading

Environment

Vuetify Version: 2.2.9
Last working version: 2.2.8
Vue Version: 2.6.11
Browsers: Chrome 80.0.3987.87
OS: Mac OS 10.14.6

Steps to reproduce

when more than one v-list is used inside a v-menu , its used to render all the lists.
from 2.2.9 version , it renders only the first list and ignores the other .And no errors are shown.
If vueitfy 2.2.8 version or lower is used , it renders all the lists with in the v-menu. Sudden change of behaviour with no error is hard to track down.Its not a blocking behaviour since code can be rewritten easily.But would like to know if this an intended behaviour or a glitch.

Expected Behavior

Till 2.2.8 , multiple v-lists were supported.Either render all the lists or show an error .

Actual Behavior

Doesnt render all the lists ,and also no errors .So its hard to track down

Reproduction Link

https://codepen.io/sar13/pen/rNVavYE

Other comments

Would like to know if the 2.2.8 behaviour is gonna be supported or should rewrite so that multiple lists are not included in the v-list
@ghost ghost added the S: triage label Feb 8, 2020
jacekkarczmarczyk added a commit that referenced this issue Feb 9, 2020
@jacekkarczmarczyk
fix(VMenu): allow more than 1 content node
f9f26cd 
fixes #10522
@jacekkarczmarczyk jacekkarczmarczyk mentioned this issue Feb 9, 2020
Merged
8 tasks
@jacekkarczmarczyk jacekkarczmarczyk added C: VMenu VMenu S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected and removed S: triage labels Feb 9, 2020
@KaelWD
Copy link
Member

KaelWD commented Feb 9, 2020

Related: cc17fe1

@jacekkarczmarczyk everything touched in 2b975f5 needs checking.

@KaelWD KaelWD added this to the v2.2.x milestone Feb 9, 2020
@KaelWD KaelWD mentioned this issue Feb 9, 2020
Closed
@jacekkarczmarczyk jacekkarczmarczyk mentioned this issue Feb 10, 2020
Closed
@BonBonSlick

This comment has been minimized.

Sign in to view
@MajesticPotatoe MajesticPotatoe mentioned this issue Feb 10, 2020
Closed
KaelWD added a commit that referenced this issue Feb 12, 2020
@jacekkarczmarczyk @KaelWD
fix(VMenu): allow more than 1 content node (#10525)
5d8d487 
* fix(VMenu): allow more than 1 content node

fixes #10522

* test

* fix(VMenu): show transitions

Co-authored-by: Kael <kaelwd@gmail.com>
@MajesticPotatoe MajesticPotatoe mentioned this issue Feb 13, 2020
Closed
@savylovskij
Copy link

When will the bug fixes [Bug Report] v-edit-dialog missing buttons after update to 2.2.12 # 10572?

@Qadosch
Copy link

Qadosch commented Feb 18, 2020 •
edited
Loading

When will the bug fixes [Bug Report] v-edit-dialog missing buttons after update to 2.2.12 # 10572?

@savylovskij
on 2.2.13 or basicaly now, i confirmed it to be working

Logikgate pushed a commit to flypapertech/vuetify that referenced this issue Feb 19, 2020
@jacekkarczmarczyk @KaelWD @Logikgate
fix(VMenu): allow more than 1 content node (vuetifyjs#10525)
f602024 
* fix(VMenu): allow more than 1 content node

fixes vuetifyjs#10522

* test

* fix(VMenu): show transitions

Co-authored-by: Kael <kaelwd@gmail.com>
whoistobias pushed a commit to whoistobias/vuetify that referenced this issue Feb 26, 2020
@jacekkarczmarczyk @KaelWD @whoistobias
fix(VMenu): allow more than 1 content node (vuetifyjs#10525)
a06c4ae 
* fix(VMenu): allow more than 1 content node

fixes vuetifyjs#10522

* test

* fix(VMenu): show transitions

Co-authored-by: Kael <kaelwd@gmail.com>
This was referenced Mar 12, 2020
Closed
Closed
Closed
Merged
@snyk-bot snyk-bot mentioned this issue Mar 29, 2020
Open
@peshanghiwa
Copy link

version 2.5.8, v-edit-dialog buttons are missing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacekkarczmarczyk jacekkarczmarczyk

Labels
C: VMenu VMenu S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
v2.2.x
7 participants
@BonBonSlick @jacekkarczmarczyk @KaelWD @saravankumarpa @Qadosch @peshanghiwa @savylovskij