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.

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

Fixes #755 - Allow diffing of HTML, head, and body. #756

Merged
merged 1 commit into from
Jul 14, 2017

Conversation

austinkelleher
Copy link
Member

Relies on the following PRs:

patrick-steele-idem/warp10#3
lasso-js/lasso#211

@austinkelleher austinkelleher force-pushed the diff-elements branch 2 times, most recently from 6f77f4b to f1e9a75 Compare July 11, 2017 13:24
@mlrawlings
Copy link
Member

I'm not sure doing this is a great idea, but it's probably the least surprising thing to support it.

@patrick-steele-idem
Copy link
Contributor

@mlrawlings

I'm not sure doing this is a great idea, but it's probably the least surprising thing to support it.

It definitely has caused some issues with folks trying marko and it was hard to explain that you need to separate out the page content from the dynamic component content. It's actually nice that the document.title will automatically update if the page re-renders in the browser.

@coveralls
Copy link

coveralls commented Jul 12, 2017

Coverage Status

Coverage increased (+0.006%) to 89.551% when pulling f1e9a75 on diff-elements into 1456f86 on master.

@coveralls
Copy link

coveralls commented Jul 13, 2017

Coverage Status

Coverage increased (+0.02%) to 89.569% when pulling acb80d2 on diff-elements into 8676705 on master.

@austinkelleher austinkelleher merged commit 126628a into master Jul 14, 2017
@DylanPiercey DylanPiercey deleted the diff-elements branch March 10, 2018 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants