lostcarpark / conclar

ConClár Programme Guide in ReactJS
MIT License
11 stars 12 forks source link

Performance Optimisation. #120

Closed lostcarpark closed 2 years ago

lostcarpark commented 2 years ago

Changes to improve performance

lostcarpark commented 2 years ago

Good spot! This was caused by the change from using locale formatting functions to just formatting hour and minute. To get the timezone code, it renders one date and extracts the timezone code. I used the epoch date of 1970 for this, which seemed to work fine for most timezones, but missed that it had problems for some, possibly if they had changed their timezone some time in the last 50 years. I've changed to using the current time to get the correct timezone code. I think this might have some problems around daylight saving changeovers, so I'll continue to look at ways of improving this.

mcdemarco commented 2 years ago

Initial load wasn’t as snappy as Peter had hoped, but performance when navigating back to the schedule is significantly improved on the Chicon schedule.

lostcarpark commented 2 years ago

Initial load wasn’t as snappy as Peter had hoped, but performance when navigating back to the schedule is significantly improved on the Chicon schedule.

Thanks for the feedback. I think there are a couple of areas we can improve things further, so I'll work on it over the next few days.