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

Convert to Preact components #458

Merged
merged 25 commits into from
Jul 25, 2019
Merged

Convert to Preact components #458

merged 25 commits into from
Jul 25, 2019

Conversation

RobbieTheWagner
Copy link
Member

This may or may not actually end up being merged, but the idea behind it is we can convert everything to Preact components, and no longer need to ship a million polyfills to support IE, if we convert things correctly. We'll see how it goes!

@genadis
Copy link
Contributor

genadis commented Jul 22, 2019

@rwwagner90 why not something like https://stenciljs.com/ ? it's tiny fast, standard, loads polyfills dynamically, works across all frameworks, easy to start with, has great tooling, has all the bells and whistles.
We had a very good experience with it with our projects. In fact when we use shepherd we first wrap it in stencil web component.

Just another alternative to consider ;)

@RobbieTheWagner
Copy link
Member Author

@genadis I mainly went with preact because it was the smallest thing I had heard of so far. I've never heard of StencilJS, but it could be a viable option. I think much of the work of splitting into a more componentized architecture will overlap with any framework, so perhaps we can finish doing Preact, and then look into something else. If you wanted to work on an alternate branch using Stencil, and mimicking the work I am doing here, we could compare the two to see which is smaller and more performant 😃

@RobbieTheWagner RobbieTheWagner changed the title [WIP] Convert to Preact Convert to Preact components Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants