Skip to content

Performance: Better bundle splitting to reduce `app.bundle.js`

Warren Gifford requested to merge tr/bundlesize into main

Created by: umpox

Actions before merging this PR:

  • Testing! Check everything looks as expected on slow connections, may need a loading spinner for certain components
  • Investigate adding bundle splitting for:
    • Remaining items in nav bar. I did this initially but reverted here. We need to change this so we always show the buttons in the nav bar, but only load the data when a button is clicked. Otherwise we end up with a flash of loading spinners in the nav bar.
    • Other items? I think with aggressive bundle splitting we can get the main bundle down to around 300kb br

Description

This PR:

  • Fixes some issues where we forgot to add bundle splitting
  • Adds bundle splitting for:
    • The references panel (as only shown on code pages)
    • Feedback prompt (only shown on click in nav)
    • Search nav bar. (avoids including fuzzy finder and other items)
    • NotepadContainer. TODO: See if this is worth doing or not

Test plan

Locally

App preview:

Check out the client app preview documentation to learn more.

Merge request reports