Remove browser-sync support

10up / 10up-toolkit

Official 10up asset building toolkit.

110 stars 20 forks source link

Remove browser-sync support #327

Open nicholasio opened 11 months ago

nicholasio commented 11 months ago

Is your enhancement related to a problem? Please describe.

Remove browser-sync support in favor of CSS Hot Reloading via HMR.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

[X] I agree to follow this project's Code of Conduct

nicholasio commented 11 months ago

It looks like browser-sync is being actively maintained again so the initial concern of outdated deps isn't a concern anymore. However I'd still like to leverage HMR for front-end css as well, but we only currently support that via a JS entrypoint on the front-end which might be not ideal for some projects.

It would be nice if we could get https://github.com/10up/10up-toolkit/issues/168 figured out so that we can have HMR as a proper replacement for css hot reloading.