Skip to content

Conversation

sergio-d-lemos
Copy link
Contributor

Fixes an issue registering window.onload JS events, also fixes a call to prettyPrint() with the wrong callback.

addEvent() doesn't seem to work well with window.onload, resulting in the registered events not being called, including the standardistaTableSortingInit() function which configures the sortable table headers.

This PR also fixes an existing error when calling prettyPrint() as the 'onload' event callback, prettyPrint expects its argument to be a callback but actually received an "event" object.

I tested on Chrome browser by navigating the Queues and Topics pages and sorting the table by different columns. Also published, moved, copied and deleted messages.

Fixes an issue registering window.onload JS events, also fixes a call to
prettyPrint() with the wrong callback.
@jbonofre jbonofre self-requested a review September 12, 2025 12:31
@jbonofre jbonofre merged commit 4352d55 into apache:main Sep 12, 2025
1 check passed
jbonofre pushed a commit that referenced this pull request Sep 12, 2025
Fixes an issue registering window.onload JS events, also fixes a call to
prettyPrint() with the wrong callback.

(cherry picked from commit 4352d55)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants