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

[Bug Report][3.6.6] VDivider vertical not visible due to height 100% resulting in 0px #19827

Closed
J-Sek opened this issue May 16, 2024 · 0 comments · Fixed by #19828 · 4 remaining pull requests
Closed

[Bug Report][3.6.6] VDivider vertical not visible due to height 100% resulting in 0px #19827

J-Sek opened this issue May 16, 2024 · 0 comments · Fixed by #19828 · 4 remaining pull requests
Assignees
Labels
C: VDivider VDivider T: bug Functionality that does not work as intended/expected
Milestone

Comments

@J-Sek
Copy link
Contributor

J-Sek commented May 16, 2024

Environment

Vuetify Version: 3.6.6
Vue Version: 3.4.27
Browsers: Chrome 124.0.0.0
OS: Linux x86_64

Steps to reproduce

  1. create flexbox container where height is controlled by it's children
  2. use <v-divider vertical> inside flexbox container

Expected Behavior

Divider should be visible and take 100% of containers height.

Actual Behavior

Divider is not visible, because height: 100% results in 0px anyway unless the container has explicit height.

Reproduction Link

https://play.vuetifyjs.com/#...

Remove style="height: auto" to make <v-divider horizontal> disappear

Other comments

I think vertical VDivider's height should be auto so that align-self: stretch applies correctly. The change does not seem to have any negative effects, however I consider this variant to always be used in the flexbox container.

@KaelWD KaelWD changed the title [Bug Report][3.6.6] VDivider horizontal not visible due to height 100% resulting in 0px [Bug Report][3.6.6] VDivider vertical not visible due to height 100% resulting in 0px May 21, 2024
@KaelWD KaelWD added T: bug Functionality that does not work as intended/expected C: VDivider VDivider labels May 21, 2024
@KaelWD KaelWD added this to the v3.6.x milestone May 21, 2024
@KaelWD KaelWD self-assigned this May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment