raphamorim / react-tv

[ Unmaintained due to raphamorim/react-ape ] React Renderer for low memory applications
https://www.npmjs.com/package/react-tv
MIT License
2.01k stars 148 forks source link

Does this work with react-hot-reload? #120

Open jfrux opened 6 years ago

jfrux commented 6 years ago

Would be nice for URL-based implementations for quick testing / debugging of the code in dev. Doing one for HiSense now and trying to get hot-reloading to work.

I've got it working, but its not actually reloading the DOM in browser.

Any tips?

raphamorim commented 6 years ago

Nope, still doesn't have it 😞 But yes, It's a "good to have" feature.

jfrux commented 6 years ago

I’m interested in contributing this feature. Before I begin; is it something deeper preventing the capability? Do you see ReactTV.render as a factor of it not working for some reason?

Is the index.html / style.css file required for ReactTV.render to work?

I’m getting an issue where if I exclude the style line from webpacl config I’m getting an error like it’s trying to include it. Even when I leave out the index.html and things from the entire equation.

Trying to decide what’s causing it.

I began pointing the entry at an index.js instead to see if I could do it that way but no dice yet.

Any advice or special considerations would be greatly appreciated! :)

I’m a 20 year web developer veteran but haven’t worked with React Renderers in this manner so didn’t know if there is something specific I’m missing.

On Apr 24, 2018, 3:18 PM -0400, Raphael Amorim notifications@github.com, wrote:

Nope, still doesn't have it 😞 But yes, It's a "good to have" feature. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.