Slow loading on larger design systems #2

Closed
opened 2024-07-14 17:44:29 -04:00 by am · 3 comments
Owner

the small set of patterns in ds-core load without too much trouble; however, I've noted on two larger design systems, it's rather slow. Consider an intersection observer to load patters as they come into view rather than all at the beginning.

the small set of patterns in ds-core load without too much trouble; however, I've noted on two larger design systems, it's rather slow. Consider an intersection observer to load patters as they come into view rather than all at the beginning.
am added the
Priority
1 High
label 2024-07-14 17:44:29 -04:00
am self-assigned this 2024-07-14 17:44:29 -04:00
Author
Owner

Observer is mostly working. Needs to unobserve otherwise patterns are getting loaded a second time. Also had to remove the rootMargin, as this was generating an error when compiling in prepros. Once we have figured out how resolve that error this can be fixed.

Observer is mostly working. Needs to unobserve otherwise patterns are getting loaded a second time. Also had to remove the rootMargin, as this was generating an error when compiling in prepros. Once we have figured out how resolve that error this can be fixed.
am added the
Kind/Enhancement
label 2024-07-15 11:53:49 -04:00
am closed this issue 2024-07-15 12:24:37 -04:00
am reopened this issue 2024-07-18 16:05:14 -04:00
Author
Owner

Attempted to fix this with Intersection observer alone didn't work. Still loads more content than is on screen.

Attempted to default articles to larger (1.5vh) didn't help. Content in two sections down was loaded and flagged as 404.

Moving to Prism.highlightAllUnder(a); doesn't seem to help, but doesn't make things worse, so probably best to have it only try to highlight the current article's code blocks.

Attempted to fix this with Intersection observer alone didn't work. Still loads more content than is on screen. Attempted to default articles to larger (1.5vh) didn't help. Content in two sections down was loaded and flagged as 404. Moving to Prism.highlightAllUnder(a); doesn't seem to help, but doesn't make things worse, so probably best to have it only try to highlight the current article's code blocks.
Author
Owner

Changed rootMargin up to 100% from half window in px ((document.body.clientHeight / 2) + "px") and added a temporary 100vh height to the articles. Changed to root margin seems to have been a huge factor.

Changed rootMargin up to 100% from half window in px ((document.body.clientHeight / 2) + "px") and added a temporary 100vh height to the articles. Changed to root margin seems to have been a huge factor.
am referenced this issue from a commit 2024-07-23 18:08:39 -04:00
am closed this issue 2024-07-23 18:08:39 -04:00
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gamv/ds2-core#2
No description provided.