-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Conversation
needs lgtm @arthurevans |
app/3.0/docs/stack-blitz.md
Outdated
index.html {.caption} | ||
``` | ||
<!-- Include WebComponents polyfills --> | ||
<script src="./node_modules/@webcomponents/webcomponentsjs/webcomponents-lite.js"></script> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going to be webcomponents-bundle.js (or webcomponents-loader.js? I think we'll be able to use the loader here). /cc: @e111077 who's been working with stackblitz, @kevinpschaaf who should know stuff just because.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think loader is more-correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought it was bundle?
app/3.0/docs/stack-blitz.md
Outdated
<!-- toc --> | ||
|
||
[StackBlitz](https://stackblitz.com) is an online editor for web projects. You can use StackBlitz to prototype and preview Polymer elements. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should add a quick summary of what StackBlitz does that other online editors don't?
In particular, it transpiles ES6 to ES5 so the samples can run on any browser we support; and that it lets you install dependencies using npm and supports node-style module resolution, so you can preview code without a build step.
You can use Polymer 3 in other online editors (like Plunker or jsbin), but the code may not be transpiled for older browsers, and you may need to use non-standard paths to import dependencies.
I might not have said that very well. But, um. I think that's why we chose it instead of using other things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few suggestions/questions, otherwise LGTM.
Not sure this belongs after the glossary, but I don't have a stronger candidate for it, either. I think I understand why you didn't put it under Tools.
because git hates me and i'm tired
incorporates all feedback from #2529
Fixes #2523