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

fix(runtime-core): avoid number type transition child key duplicating with index key #5779

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix(runtime-core): avoid number type transition child key duplicating with index key #5779

wants to merge 2 commits into from

Conversation

yuwu9145
Copy link
Contributor

@yuwu9145 yuwu9145 commented Apr 22, 2022 •
edited

close #5761

@netlify
Copy link

netlify bot commented Apr 22, 2022 •
edited

✅ Deploy Preview for vue-sfc-playground ready!

Name Link
🔨 Latest commit 0050a37
🔍 Latest deploy log https://app.netlify.com/sites/vue-sfc-playground/deploys/626784d925b6d200086be919
😎 Deploy Preview https://deploy-preview-5779--vue-sfc-playground.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 22, 2022 •
edited

✅ Deploy Preview for vue-next-template-explorer ready!

Name Link
🔨 Latest commit 0050a37
🔍 Latest deploy log https://app.netlify.com/sites/vue-next-template-explorer/deploys/626784d9e9d1ed0009a1e475
😎 Deploy Preview https://deploy-preview-5779--vue-next-template-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Apr 22, 2022 •
edited

✅ Deploy Preview for vuejs-coverage ready!

Name Link
🔨 Latest commit 0050a37
🔍 Latest deploy log https://app.netlify.com/sites/vuejs-coverage/deploys/626784d900dba500088a5c3a
😎 Deploy Preview https://deploy-preview-5779--vuejs-coverage.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@yuwu9145 yuwu9145 force-pushed the fix-5762 branch 2 times, most recently from 45830e3 to 74cddb5 Compare April 22, 2022 04:52
@yuwu9145 yuwu9145 changed the title fix(runtime-core): respect child index position in transition group w… fix(runtime-core): respect child index position when calculating key Apr 22, 2022
@yuwu9145 yuwu9145 force-pushed the fix-5762 branch 2 times, most recently from 7e530b5 to 9f4facb Compare April 22, 2022 11:24
@yuwu9145 yuwu9145 changed the title fix(runtime-core): respect child index position when calculating key fix(compiler-core): v-if key increment also based on other type siblings Apr 22, 2022
@yuwu9145
Copy link
Contributor Author

I will add tests soon

@yuwu9145 yuwu9145 force-pushed the fix-5762 branch 2 times, most recently from dad9798 to cc93966 Compare April 22, 2022 12:10
@yuwu9145 yuwu9145 mentioned this pull request Apr 22, 2022
Open
@yuwu9145 yuwu9145 changed the title fix(compiler-core): v-if key increment also based on other type siblings fix(runtime-core): avoid number type transition child key duplicating with index key Apr 22, 2022
@yuwu9145 yuwu9145 force-pushed the fix-5762 branch 2 times, most recently from ea78b49 to 0050a37 Compare April 26, 2022 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
Issue / PR Management
Status: Stale
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Duplicate key for transition group fragment when child has v-if
1 participant
@yuwu9145