Skip to content

fix(dev): disable hoisted for dev#521

Closed
underfin wants to merge 1 commit into
masterfrom
dev-disable-hoisted
Closed

fix(dev): disable hoisted for dev#521
underfin wants to merge 1 commit into
masterfrom
dev-disable-hoisted

Conversation

@underfin
Copy link
Copy Markdown
Member

@underfin underfin commented Jul 6, 2020

close #514

Bug Reason

Because hoisted vnode only have one copy, the el will caused conflict with multi used.But it is a design, the static vnode will not update forever on production.So we need diasbled hoisted in dev.

@yyx990803
Copy link
Copy Markdown
Member

See vuejs/core@5f61aa0

I think it's better to use the same compiler options for dev/prod to ensure behavior consistency.

@yyx990803 yyx990803 closed this Jul 15, 2020
@underfin underfin deleted the dev-disable-hoisted branch July 16, 2020 01:36
cpojer pushed a commit to cpojer/vite that referenced this pull request Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When using duplicates of a component, only the first one updates

2 participants