mntnr / dashboard

A status board for repositories within a GitHub organization.
https://maintainer.io
Other
21 stars 4 forks source link

Move away from BrowserSync #18

Open RichardLitt opened 6 years ago

RichardLitt commented 6 years ago

If we want to make this work for a hosted webapp, I'm not sure BrowserSync is the best option. Alternatively, learn more.

tgetgood commented 6 years ago

So brower sync is a dev and testing tool. It allows you to control multiple browsers and device emulators (and devices) as slaves chained to a single master. Thus you can fill in a form once and see it submit on a dozen platforms. It's pretty cool actually. It doesn't have anything to do with production or deployment --- it's currently being used in production, but that's not essential, probably not a great idea even --- but we could leave it for dev or not as we choose.

What are our requirements for cross browser and device compatibility / responsiveness? If they're not very important then browser sync is overkill. My impression is that this isn't some cutting edge spa that needs tweaks for each browser, it's a dashboard, so compatibility shouldn't be much of a problem.

Browser sync is also giving us hotloading for js and css, but we can get that from other sources too.

RichardLitt commented 6 years ago

Leaving it for dev would work for me. I don't have any requirements for cross-browser, I tend to work solely in the desktop.

Let's get another hotloading option.