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

types: relax TextareaHTMLAttributes.value type requirement #5161

Closed
wants to merge 1 commit into from

Conversation

cexbrayat
Copy link
Member

Similar to 93949ed and 6171aec but this time to allow textarea to have a v-model with a null value for example.

@cexbrayat cexbrayat changed the title types: relax TexteareaHTMLAttributes.value type requirement types: relax TextareaHTMLAttributes.value type requirement Dec 23, 2021
@cexbrayat
Copy link
Member Author

Rebased on top of #5162 to have a proper CI result

Similar to 93949ed and 6171aec but this time to allow `textarea` to have a `v-model` with a null value for example.
@sxzz sxzz force-pushed the fix/textarea-null-v-model branch from 2cfd86f to 9d9267c Compare August 21, 2023 08:02
@sxzz sxzz added the ready for review This PR requires more reviews label Aug 21, 2023
@github-actions
Copy link

Size Report

Bundles

File Size Gzip Brotli
runtime-dom.global.prod.js 85.8 kB 32.6 kB 29.4 kB
vue.global.prod.js 132 kB 49.3 kB 44.3 kB

Usages

Name Size Gzip Brotli
createApp 47.7 kB 18.9 kB 17.2 kB
createSSRApp 50.5 kB 19.9 kB 18.2 kB
defineCustomElement 50.1 kB 19.6 kB 17.9 kB
overall 61.1 kB 23.7 kB 21.6 kB

@cexbrayat
Copy link
Member Author

This is no longer necessary now that #9997 has been merged

@cexbrayat cexbrayat closed this Jan 8, 2024
@cexbrayat cexbrayat deleted the fix/textarea-null-v-model branch January 8, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR requires more reviews
Projects
Status: Rejected
Development

Successfully merging this pull request may close these issues.

None yet

2 participants