[go: up one dir, main page]

Move blob info request to startup JS

What does this MR do and why?

Moves the GraphQL request for blob info to startup JS as part of #366531 (closed)

This will ensure some of the LCP performance that we'll lose when highlighting the entire file is offset by the LCP improvement we get from changes in this MR.

MR breakdown

Description MR
Move the blob_info GraphQL request to StatupJS 👈 this MR
Create a WebWorker to handle the task of Highlighting !98340 (merged)
Highlight all source code with the WebWorker follow-up

Screenshots or screen recordings

before after
large GraphQL request is made towards the end of the timeline large GraphQL request is made towards the beginning of the timeline
*slight improvement in LCP
Screenshot_2022-09-08_at_15.06.52 Screenshot_2022-09-08_at_15.07.47

How to set up and validate locally

  1. Open a text file from the repository files viewer
  2. Run a performance audit
  3. Notice the GraphQL request is made towardss the beginning of the timeline

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #366531 (closed)

Edited by Jacques Erasmus

Merge request reports

Loading