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

Using arrays in style can cause errors in some cases #5106

Closed
sushoucanyuan opened this issue Dec 14, 2021 · 1 comment
Closed

Using arrays in style can cause errors in some cases #5106

sushoucanyuan opened this issue Dec 14, 2021 · 1 comment
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. 🐞 bug Something isn't working scope: compiler

Comments

@sushoucanyuan
Copy link

Version

3.2.26

Reproduction link

sfc.vuejs.org/

Steps to reproduce

Delete a div that is not the last div

What is expected?

work well

What is actually happening?

throw an error.
Failed to set an indexed property on 'CSSStyleDeclaration': Indexed property setter is not supported.
@lidlanca lidlanca mentioned this issue Dec 14, 2021
Closed
@lidlanca
Copy link
Contributor

for a static vnode chunk created by the compiler createStaticVNode styles are being merged staticall if the input is an array of objects.

for the none static case(patchStyle) does not have handling logic for array of objects

@yyx990803 yyx990803 added 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. 🐞 bug Something isn't working scope: compiler labels Jan 16, 2022
@baiwusanyu-c baiwusanyu-c mentioned this issue May 10, 2022
Closed
baiwusanyu-c added a commit to baiwusanyu-c/bwsy-vue-core that referenced this issue May 10, 2022
@baiwusanyu-c
Revert "fix(runtime-dom): Fixed a runtime error when the element node…
cd4b15c 
…'s style binding array was not compiled as 'createStaticVNode'. (fix vuejs#5106)"
baiwusanyu-c added a commit to baiwusanyu-c/bwsy-vue-core that referenced this issue May 10, 2022
@baiwusanyu-c
Revert "fix(runtime-dom): Fixed a runtime error when the element node…
8c1e3fa 
…'s style binding array was not compiled as 'createStaticVNode'. (fix vuejs#5106)"
baiwusanyu-c added a commit to baiwusanyu-c/bwsy-vue-core that referenced this issue May 11, 2022
@baiwusanyu-c
fix(compiler-core): Fix the 'constType' error when using 'v-bind' to …
747a2ab 
…bind array variables to 'style', resulting in runtime errors(fix vuejs#5106)
@baiwusanyu-c baiwusanyu-c mentioned this issue May 11, 2022
Closed
baiwusanyu-c added a commit to baiwusanyu-c/bwsy-vue-core that referenced this issue May 11, 2022
@baiwusanyu-c
fix(compiler-core): Fix the 'constType' error when using 'v-bind' to …
2f144a1 
…bind array variables to 'style', resulting in runtime errors(fix vuejs#5106)
iwusong pushed a commit to iwusong/core that referenced this issue May 13, 2022
@yyx990803 @iwusong
fix(compiler-core): normalize v-bind:style with array literal value
0cf83ea 
fix vuejs#5106
@github-actions github-actions bot locked and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. 🐞 bug Something isn't working scope: compiler
Projects
None yet
Milestone
No milestone
3 participants
@yyx990803 @lidlanca @sushoucanyuan