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][3.6.4] Default prop values with a null value are ignored #19783

Closed
donalmurtagh opened this issue May 8, 2024 · 0 comments · Fixed by #19842 · 4 remaining pull requests
Closed

[Bug Report][3.6.4] Default prop values with a null value are ignored #19783

donalmurtagh opened this issue May 8, 2024 · 0 comments · Fixed by #19842 · 4 remaining pull requests
Assignees
@KaelWD
Labels
E: defaults Defaults composable T: bug Functionality that does not work as intended/expected
Milestone
v3.6.x

Comments

@donalmurtagh
Copy link
Contributor

donalmurtagh commented May 8, 2024 •
edited
Loading

Environment

Vuetify Version: 3.6.4
Vue Version: 3.4.27
Browsers: Chrome 124.0.0.0
OS: Mac OS 10.15.7

Steps to reproduce

Remove the mobile="null" prop value from the data table and wait for the demo to reload

Expected Behavior

The table should stay in mobile mode, because mobile="null" is the default value for this prop in vuetify.js

Actual Behavior

The table switches to desktop mode (presumably) because default props with a value of null don't work

Reproduction Link

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

Comment

I tried setting the default value to undefined instead of null, but that also didn't work.
@johnleider johnleider added T: bug Functionality that does not work as intended/expected E: defaults Defaults composable and removed S: triage labels May 13, 2024
@KaelWD KaelWD added this to the v3.6.x milestone May 21, 2024
@KaelWD KaelWD linked a pull request May 21, 2024 that will close this issue
fix(defaults): allow null component and global values #19842
Merged
KaelWD pushed a commit that referenced this issue May 21, 2024
@webdevnerdstuff
fix(defaults): allow null component and global values (#19842)
9342288 
fixes #19783
@aniketmaurya aniketmaurya mentioned this issue Jun 11, 2024
Closed
@YoutacRandS-VA YoutacRandS-VA mentioned this issue Jun 19, 2024
Open
@aniketmaurya aniketmaurya mentioned this issue Jun 20, 2024
Closed
@aniketmaurya aniketmaurya mentioned this issue Jul 3, 2024
Closed
@aniketmaurya aniketmaurya mentioned this issue Jul 11, 2024
Closed
This was referenced Jul 16, 2024
Open
Open
@aniketmaurya aniketmaurya mentioned this issue Jul 21, 2024
Closed
@snyk-io snyk-io bot mentioned this issue Jul 24, 2024
Open
@Asthriona Asthriona mentioned this issue Aug 7, 2024
Closed
@WontonSam WontonSam mentioned this issue Aug 16, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KaelWD KaelWD

Labels
E: defaults Defaults composable T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
v3.6.x
3 participants
@johnleider @donalmurtagh @KaelWD