pfSense

First of all thank you for looking at the PR and your commits. I agree with all the changes you've made to my commits, I think it's better this way.

Regarding the rebuilding -

While looking at my commit it occurs to me that I probably didn't mention a very important detail: the variable firstTime doesn't work correctly and the buildThermalSensorsDataGraph function is the one that's executed on every ajax call. updateThermalSensorsDataGraph never gets executed. This generates unnecessary load on clients and slows down the refresh.
Maybe it would be better to fix the issue with the variable first. This issue is present in some other widgets as well. I can't remember all the details since the PR is almost 2 years old.

I'd like to contribute more to the widget refresh system/widgets code, however it's hard to track things that are as old as this. I have some other commits written, but I'd like to ask if it's possible for you to review the code a bit sooner (a couple of weeks, a month). I know you're not obligated to do anything here, but I'd like to know if my commits will make it into the project and ultimately, if it's worth my time as well.

For example, I have a simple fix for the widget refresh system as I think the "refresh" doesn't work as intended at the moment (refresh interval is bugged, and widgets don't all update at the same time). I'll take a look at the code, clean it up and submit a PR. Maybe the No timeout on first load commit can go there as well.

Let me know what you think of the html rebuilding and the firstTime variable.
Thank you!

I agree it's slow. My only advice is to keep the commits focused; the more detail/explanation about why a change is made the better.

Yes, I will look at it sooner - feel free to tag me in the PR.

Thank you - I've created another PR for the widget refresh, and I'll put optimizations in yet another PR.
You can close this redmine issue now.