bug: `diff` children, the vnodes can includes `null` node
See original GitHub issueDescribe the bug A clear and concise description of what the bug is.
so, the element can be removed, it’s not work as excepted.
if we filter the vnodes
in my demo, it work perfect.
but, this causes the order of vnodes to be out of order, i try to find a nother way to resolve it. what’s your opinion? To Reproduce Steps to reproduce the behavior:
- Go to ‘…’
- Click on ‘…’
- Scroll down to ‘…’
- See error
Expected behavior A clear and concise description of what you expected to happen.
Screenshots If applicable, add screenshots to help explain your problem.
Device (please complete the following information):
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
Additional context Add any other context about the problem here.
Issue Analytics
- State:
- Created a year ago
- Comments:6 (4 by maintainers)
Top Results From Across the Web
Minor Feature: Normalize vnode.children passed to m ...
vnodes : normalize boolean children to null/undefined at the vnode level, ... diff where each node has a key set to the vnode's...
Read more >VNode - IBM
The VNode interface defines the representation of an system resource that exists in a hierarchy. In the hierarchy, a node has exactly one...
Read more >Vue error makes no sense: Unhandled error during execution ...
This happens because the PostCreate view is trying to recursively load itself. Since you used the same name for the view and the...
Read more >vnode.h - Apple Open Source
When a RESOLVER_ERROR * occurs, VFS will propagate this error back to the ... data The arbitrary data supplied by vnode trigger creator...
Read more >Preact 5 - Releases
Features. When rendering null , a comment node will be inserted instead of ... Fix IE bug caused by assigning to the nodeValue...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I’ll be adding some more fixed to this PR to fix #203 – expect a merge by the end of this week!
I see, I’ll make some changes and merge your PR. Thanks for the explanation