Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

docs: preloads docs assets #4384

Merged
merged 5 commits into from
Jul 5, 2023
Merged

docs: preloads docs assets #4384

merged 5 commits into from
Jul 5, 2023

Conversation

davidmurdoch
Copy link
Member

@davidmurdoch davidmurdoch commented May 5, 2023

Reduces page load time by initiating downloads of assets that will get loaded from other CSS and JS files later.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented May 5, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9cd0be3
Status: ✅  Deploy successful!
Preview URL: https://60d01fea.ganache.pages.dev
Branch Preview URL: https://docs-preload.ganache.pages.dev

View logs

@davidmurdoch davidmurdoch marked this pull request as ready for review May 6, 2023 00:35
docs/assets/js/inject-editor.js Outdated Show resolved Hide resolved
scripts/build-docs/index.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved! Just a suggestion on an errant comma.

scripts/build-docs/index.ts Outdated Show resolved Hide resolved
Co-authored-by: jeffsmale90 <6363749+jeffsmale90@users.noreply.github.com>
@davidmurdoch davidmurdoch merged commit 2969bc7 into develop Jul 5, 2023
@davidmurdoch davidmurdoch deleted the docs-preload branch July 5, 2023 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants