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

fix(useDisplay): keep default behavior when used without args #20935

Merged
merged 1 commit into from
Jan 29, 2025
Merged

fix(useDisplay): keep default behavior when used without args #20935

merged 1 commit into from
Jan 29, 2025

Conversation

J-Sek
Copy link
Contributor

@J-Sek J-Sek commented Jan 28, 2025 •
edited
Loading

Description

follow-up to #20311
and alternative to #20930

resolves #20932

Markup:

const { mobile } = useDisplay()
console.log({ mobile }) // used to work before v3.7.8, now it is always false

// workaround: useDisplay({ mobile: null })

@johnleider johnleider added T: bug Functionality that does not work as intended/expected T: regression Something that used to work but we broke E: display Display composable and removed T: regression Something that used to work but we broke labels Jan 29, 2025
@johnleider johnleider merged commit bb8b3f8 into vuetifyjs:master Jan 29, 2025
10 checks passed
@J-Sek J-Sek deleted the fix/use-display branch January 29, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnleider johnleider johnleider approved these changes

Assignees

@J-Sek J-Sek

Labels
E: display Display composable T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug Report][3.7.8] useDisplay() composable no longer returning mobile correctly
2 participants
@J-Sek @johnleider