Conditionals in looped <template> prevent previous items from being output

[bradisbell]

1. Describe your issue:

If using <template each="...">, the items inside <template> override each other in certain conditions:

• When using text only (no non-text child elements of <template>)
• When using conditional elements (like <p if="...">{something}</p>

2. Can you reproduce the issue?

Yep!

https://plnkr.co/edit/BkD29KjxRdXtmy5BO3ub?p=preview

Change template to div and you'll see the differences.

3. On which browser/OS does the issue appear?

Chrome v77 x64 on Windows 10

4. Which version of Riot does it affect?

v4

5. How would you tag this issue?

Bug

[GianlucaGuarini]

@bradisbell your example seems to me an edge case since your html could be rewritten in a cleaner and reusable way
I will have a look at this issue but I will classify it as a low prio.
Thanks for reporting it

[bradisbell]

The example was the most trivial example I could think of, to make it easy to reproduce the bug in Riot.js. The reality in my application has a lot more to it, and the way it is structured makes sense for my application in-context.

There is also the text case, which isn't affecting me at the moment but I thought I'd point it out as it was unexpected.

Anyway, I'm able to use extra divs for now.

[GianlucaGuarini]

Fixed in Riot.js@4.6.6