Load extension JavaScript bundles before setting up a page.

Review Request #14610 — Created Sept. 11, 2025 and submitted

Information

Review Board
release-7.1.x

Reviewers

When loading a page, we were setting up the Page and PageView state
before loading extension JavaScript bundles. This meant that any page
that wanted to customize the page had to essentially break this bit of
code out into a separate bundle and then manually invoke loading the
bundle ahead of time, which is far from ideal.

We now simply ensure we've loaded this JavaScript before-hand.

This could potentially break some extensions if they assume that the
page is set up first, but JavaScript is always supposed to use
RB.PageManager.beforeRender() or RB.PageManager.ready() for any
page-specific JavaScript code, to be safe.

Tested loading a new Power Pack-provided PageView, and verified it
worked.

Summary ID
Load extension JavaScript bundles before setting up a page.
When loading a page, we were setting up the `Page` and `PageView` state before loading extension JavaScript bundles. This meant that any page that wanted to customize the page had to essentially break this bit of code out into a separate bundle and then manually invoke loading the bundle ahead of time, which is far from ideal. We now simply ensure we've loaded this JavaScript before-hand. This *could* potentially break some extensions if they assume that the page is set up first, but JavaScript is always supposed to use `RB.PageManager.beforeRender()` or `RB.PageManager.ready()` for any page-specific JavaScript code, to be safe.
a66e13cfc3919a591415dfa484dcf82d4edaf9ba
maubin
  1. Ship It!
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-7.1.x (cb3cb67)