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 v-show and :style="display: 'block'" simultaneously does not yield the expected results in version 2.7.16. #13140

Open
senwong opened this issue Dec 28, 2023 · 5 comments · May be fixed by #13198
Open

Using v-show and :style="display: 'block'" simultaneously does not yield the expected results in version 2.7.16. #13140

senwong opened this issue Dec 28, 2023 · 5 comments · May be fixed by #13198

Comments

@senwong
Copy link

senwong commented Dec 28, 2023

Version

2.7.16

Reproduction link

codesandbox.io

Steps to reproduce

2.7.16 Demo

when the button is clicked, it initiates a component update. Subsequently, the content div is meant to be hidden. Nevertheless, the current behavior deviates, as the content div is displayed rather than being hidden.

2.7.15 Demo
In version 2.7.15, clicking the button triggers a component update, and as a result, the content div consistently remains hidden.

The version 2.7.16 should not introduce this breaking change.

What is expected?

v-show should have a higher priority than :style.

What is actually happening?

The style is being updated incorrectly.
@dkdk135
Copy link

dkdk135 commented Jan 4, 2024

amazing

@carakas
Copy link

carakas commented Jan 8, 2024

Ran into this as well, had to revert to v2.7.15 after receiving a complaint

@ZAID-BAARAB
Copy link

i never noticed that !

1 similar comment
@zhongzi-rail0039
Copy link

i never noticed that !

@jackiotyu
Copy link

The error comes from: f5ef882

@nieyuyao nieyuyao linked a pull request Jul 9, 2024 that will close this issue
Open
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(v-show): determine v-show when updating styles nieyuyao/vue
6 participants
@carakas @dkdk135 @zhongzi-rail0039 @senwong @jackiotyu @ZAID-BAARAB