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.

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

Fix disappearing HTML elements, Fixes #862 #79

Merged
merged 1 commit into from Jan 6, 2015

Conversation

Projects
None yet
2 participants
@jwmerrill
Contributor

jwmerrill commented Jan 5, 2015

See discussion on #862, especially elm/compiler#862 (comment).

This merits careful review because I don't know why the kid.getContext check that I'm removing was there in the first place. @etaque reports that this change fixes the issue.

Fix disappearing HTML elements, Fixes #862
See discussion on #862, especially elm/compiler#862 (comment).

This merits careful review because I don't know why the `kid.getContext` check that I'm removing was there in the first place. @etaque reports that this change fixes the issue.
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Jan 6, 2015

Member

Reading the code and knowing what my intent was with that code, I am pretty sure this is just a bug. I have not touched this code in ages, so it is somewhat mysterious if this arose with 0.14. In any case, this looks good to me!

Thank you so much for figuring this out, I really really appreciate it!

Member

evancz commented Jan 6, 2015

Reading the code and knowing what my intent was with that code, I am pretty sure this is just a bug. I have not touched this code in ages, so it is somewhat mysterious if this arose with 0.14. In any case, this looks good to me!

Thank you so much for figuring this out, I really really appreciate it!

evancz pushed a commit that referenced this pull request Jan 6, 2015

Merge pull request #79 from jwmerrill/patch-2
Fix disappearing HTML elements, Fixes #862

@evancz evancz merged commit de5d8e0 into elm:master Jan 6, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment