Skip to content

Conversation

@FWeinb
Copy link
Contributor

@FWeinb FWeinb commented Dec 4, 2016

In preparation for {{yield}} I added a node type of YieldTag. So it is possible to extend the yield feature to named yields in the future.

What is working:

  • yield block is updated by parent components scope
  • yield is passed to the child
  • teardown and mount is handled
  • yield is working in {{#if}}
  • generator should complain when yield is in {{each}}??
  • merge with move component codegen into its own file #111??
* sveltejs/master: -> v1.0.7 use decodeCharacterReferences in text.js, not tag.js -> v1.0.6 upgrade magic-string, fix sourcemaps add .json extension to the ESLint config file add links to gulp/Metalsmith plugins more sourcemap stuff Convert entities of all text nodes add infrastructure for sourcemap tests Adding link to Browserify transform dont use options.filename for original code when generating magicstring bundle (#43)
* sveltejs/master: keep target and anchor inside mount method rename parent to root Fix typo in generator's contextualise; solves #46 fix onrender hook for nested widgets add a createAnchor helper various refactorings & fixes separate create from mount
@codecov-io
Copy link

codecov-io commented Dec 4, 2016

Current coverage is 91.00% (diff: 100%)

Merging #112 into master will increase coverage by 0.14%

@@ master #112 diff @@ ========================================== Files 48 49 +1 Lines 1324 1345 +21 Methods 0 0 Messages 0 0 Branches 0 0 ========================================== + Hits 1203 1224 +21  Misses 121 121 Partials 0 0 

Powered by Codecov. Last update 5770048...cc7c92a

@FWeinb FWeinb changed the title Add YieldTag to parser Add YieldTag to parser and generator Dec 4, 2016
@FWeinb
Copy link
Contributor Author

FWeinb commented Dec 5, 2016

I think I now got the code generation right.

@Rich-Harris Rich-Harris merged commit cc7c92a into sveltejs:master Dec 5, 2016
@Rich-Harris
Copy link
Member

This looks great, thanks @FWeinb – have gone ahead and merged it because #111 has some conflicts that need to be resolved separately anyway

Rich-Harris added a commit that referenced this pull request Dec 5, 2016
@Swatinem Swatinem mentioned this pull request Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants