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.3.12] Missing box-shadow for Scrollable Dialog #18052

Closed
dmcknight26 opened this issue Aug 14, 2023 · 1 comment
Closed

[Bug Report][3.3.12] Missing box-shadow for Scrollable Dialog #18052

dmcknight26 opened this issue Aug 14, 2023 · 1 comment
Assignees
Labels
C: VDialog VDialog T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke
Milestone

Comments

@dmcknight26
Copy link

Environment

Vuetify Version: 3.3.12
Last working version: 3.3.11
Vue Version: 3.3.4
Browsers: Chrome 115.0.0.0
OS: Windows 10

Steps to reproduce

Create a scrollable popup and open it.

Expected Behavior

There should be a box-shadow around the dialog.

Actual Behavior

There is no box-shadow.

Reproduction Link

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

Other comments

This appears to be happening because of a recently introduced change to the v-overlay__content parent element in the DOM, which now has overflow: hidden when in scrollable mode which suppresses the box-shadow on the dialog.

@KaelWD KaelWD added T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke C: VDialog VDialog labels Aug 15, 2023
@KaelWD KaelWD self-assigned this Aug 15, 2023
@cll0ud
Copy link

cll0ud commented Sep 26, 2023

Using this CSS as a workaround for now:

.v-dialog--scrollable > .v-overlay__content {
  overflow: initial
}

@KaelWD KaelWD added this to the v3.4.x milestone Nov 14, 2023
@KaelWD KaelWD closed this as completed in 3ce876f Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VDialog VDialog T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke
Projects
None yet
Development

No branches or pull requests

3 participants