sergio-d-lemos opened a new pull request, #1479:
URL: https://github.com/apache/activemq/pull/1479

   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](https://github.com/apache/activemq/blob/2eb0d6675008a5e7fd6cbec4a177ab13fde3b8f2/activemq-web-console/src/main/webapp/js/standardista-table-sorting.js#L424C10-L424C38)()
 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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org
For additional commands, e-mail: gitbox-h...@activemq.apache.org
For further information, visit: https://activemq.apache.org/contact


Reply via email to