Skip to content
This repository has been archived by the owner on Apr 4, 2019. It is now read-only.

[WIP] Rehydration #404

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[WIP] Rehydration #404

wants to merge 6 commits into from

Conversation

wycats
Copy link
Contributor

@wycats wycats commented Aug 18, 2015

This WIP PR implements rehydration at the HTMLBars level.

There are a few things we need to do before we can consider the primitive level "complete":

  • templateId (done in db6462f)
  • MorphList
  • Multiple adjacent text nodes (done in a09bed6)
  • Empty text nodes
  • Dynamic templates (aka ManualElement)

@mmun
Copy link
Collaborator

mmun commented Aug 18, 2015

🎉

Godhuda added 3 commits August 18, 2015 16:28
This code path predates the existence of `blockFor`, which is a more
general-purpose and flexible mechanism for implementing Ember-style
“layouts” that `{{yield}}` to the block provided to a component.
This allows us to use a unified code path between rehydration and normal
rendering.

We also discovered that we don’t use `lastYielded.self` (it was an
accidental holdover from the code that was refactored into lastYielded),
so we removed it (along with a vestigial `shadowTemplate`, removed in a
previous commit).
Godhuda added 2 commits August 18, 2015 17:40
This is important because we use empty text nodes as boundary nodes.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants