Skip to content

Commit

Permalink
#22 #23 #24 #25 Moved README TODOs to GitHub issues
Browse files Browse the repository at this point in the history
  • Loading branch information
hexus committed May 17, 2021
1 parent 53f5c03 commit 4764a6e
Showing 1 changed file with 0 additions and 36 deletions.
36 changes: 0 additions & 36 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,39 +27,3 @@ Build and watch for changes:
```bash
$ npm run watch
```

## TODO

### Bundle runtime separately

The Pragma runtime (primarily `FormProcessor`) is only bundled as a side effect of the example project.

Bundle it separately from the example project after clarifying the shape of Pragma's public API.

### Bundle components with Webpack

Stencil currently [only bundles using Rollup](https://stenciljs.com/docs/module-bundling).

With better [Public Compiler APIs](https://stenciljs.com/blog/stencil-roadmap-fall-2019#public-compiler-apis) it should
instead be possible to compile Stencil components and bundle them as part of Webpack, instead of needing to run two
watches. For now, the two watches are seemingly necessary.

**Update**: The [Stencil Core Compiler API](https://stenciljs.com/docs/compiler-api) is now documented, but there does
not yet appear to be a Webpack plugin that utilises it.

### Experiment with extending built-in elements

Web components can extend built-in HTML elements to inherit functionality and usability.

Stencil doesn't currently support this in any way, but a keen-eyed developer has found a workaround:
- https://github.com/ionic-team/stencil/issues/1382#issuecomment-833095324

Try this out and see if the boilerplate repetition in the Pragma components can be reduced.

**Note:** This approach doesn't currently include form participation, but the same comment will be updated when the
developer gets around to it.

### Safer concurrent watches

Consider using [Concurrently](https://github.com/kimmobrunfeldt/concurrently) for the concurrent Webpack & Stencil
watches. The watch script is currently relying on bash's `&`.

0 comments on commit 4764a6e

Please sign in to comment.