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

Nested v-show ignores 'outer' value, and component with v-show="false" is shown. #10338

Closed
MeesterDev opened this issue Feb 14, 2024 · 3 comments
Labels

Comments

@MeesterDev
Copy link

Vue version

3.4.19

Link to minimal reproduction

https://play.vuejs.org/#eNqNVNtu4jAQ/ZWRX0olSEDsy7IJ6u6qD11pL+rymJcQD+DWsSPbCUiUf+/YARpuUq0oiudy5pyZ2Fv2vaqipkY2YYnDspK5w2mmgFbypBSan7qstELloBnYlV6nGVvk0mLGyCAWtLWOciykaQpDeHuDzn5EUTHBJXEHm7a2MKJyYNHVFchcLQnG2YztKwsqaRxsn3HRB+NfWv3WtXLI+6Bw42aieN3BwugS7oj83beTvDPebVgUn5q9aJ/XZhZaWeITmE+A6iYkBUZTSH393vD+GHlk0uvdT6DRgkM6hW3r9KsFiZpc1kjpo2OmXwfyt5KvAAz34vzaeR7HjyRu+0hdY33Wih+UeRW9WK1ooAE32zuou5NDpYyReL/P2Mq5yk7iuOCK0jhK0ZhIoYtVVcYPFBYbUitKHHBdPoyjL9Hoa8yFdV17hLYczI1eWzSEkrF+p05MxgbNwKDiaNB8tu5Z2kntM99FfV9+l6kdtcVZmu1CLM+aUtB/ICSav5UTNPuT5uRS6vWvYHOmxqOYYoXF6xX7i920ov4ZDMw6DXC5WaJr3Y///9D4O85S81ruB3HD+YxWy9pzbMN+1IoT7U5cYPsUZizUcmYfNw6VPYjyREM3QnwYiD8Ct6R/0B1H404XL8/OtQuDi+bjlgiV2XS2EhbocSsERwpBqPAtPCKduwMkzC6NPk8hTRo8puqD1ZTrwTwQ2WrJ9wFzhEZYMZcYJfSPNKeXDtu9AyiSnu4=

Steps to reproduce

Open the link, the text of the inner component is immediately shown.

In 3.4.15 and 3.4.17, the text is not shown.

What is expected?

The text should not be shown, as the most top-level v-show has a "false" value.

What is actually happening?

The text is shown.

System Info

Reproducable at least in: Firefox 122.0, Firefox Developer 123.0b4, and Chrome 121.0.6167.161. 64bit machine.

[I don't think this is relevant for this bug, but who knows]

Any additional comments?

No response

@edison1105
Copy link
Member

edison1105 commented Feb 15, 2024

related to #10311

/cc @zh-lx

@edison1105 edison1105 added 🐞 bug Something isn't working regression labels Feb 15, 2024
@Doctor-wu
Copy link
Member

Doctor-wu commented Feb 15, 2024

In this case two vShow directive was applied to the same element, I'm not sure if it's a good practice, should we just merge two vShow directives (If do so, I prefer to respect the parent vShow), or we need to maintain separate state for each vShow directive? see #10344

@yyx990803
Copy link
Member

Closed by mistake

Doctor-wu added a commit to Doctor-wu/core that referenced this issue Feb 25, 2024
@yyx990803 yyx990803 added 🔩 p2-edge-case and removed 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. labels Feb 25, 2024
OnlyWick pushed a commit to OnlyWick/core that referenced this issue Feb 27, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 12, 2024
lynxlangya pushed a commit to lynxlangya/core that referenced this issue May 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants