search

magnifydev / magnify

A Digital Program of Studies for LMHS.
https://magnify.web.app
Apache License 2.0
8 stars 0 forks source link

Implement lazy loading of course elements and data #52

Closed GoogolGenius closed 1 year ago

GoogolGenius commented 2 years ago

The website is horrendous on slower devices lol

Ascent817 commented 2 years ago

I think lazy loading is nice, but it isn't possible with the way the code was written(badly). We'd need to do a significant refactor of the way that the courses are loaded for it to be possible.

GoogolGenius commented 2 years ago

I guess this will be backlogged. I think the priority now is on refactoring out the components.

GoogolGenius commented 2 years ago

Right now the components are loaded by looping through the data essentially and creating a component element for each part of the course data we receive. I also was thinking about an implementation detail that could have been overlooked, which is that when trying to jump down to a particular element (if we were to add that feature, which I think I will sometime soon), that may be an issue because the element isn't loaded yet, therefore not having the respective id tag which we could update the browser URL maybe.