I see. I didn’t actually see the patch since I didn’t go so far as digging into the jinjor/elm-break-dom
project to look for it. I wonder why this patch was never submitted as a PR to elm/virtual-dom
? Perhaps the author thought there was no point as it would not be merged?
Only comment I could find from Evan on the issue is:
https://github.com/elm/html/issues/44#issuecomment-233145854
In the past, people (I believe including @rtfeldman) expressed frustration with having that additional named div in there, so I took it out. So I’m surprised by this recommendation. I’m not sure what to do given the history of this.
(the extra div approach screws up printer layouts apparently)
I will change the status of this one to “Runtime Exception” - seems like the patch could be a straightforward bug fix and a really helpful one too.
==============
Found some more from Evan:
Possibly that post got jinjor started on compiling a list of problems and coming up with a fix. We just never completed the process and got the fix out…