Change Details

Duration: 8hrs Let's review the existing JavaScript code before pushing to stable. **Analyze**: [] How does it impact first render time / fully loaded load time [] How long does a hovercard take to load (especially as @nuria points out on slow connections) [] Which libraries does it use and which are loaded asynchronously * ~~Review Icon support to see if it will benefit from ResourceLoaderImageModule~~ T133956 * ~~ext.popups.settings.js would benefit from template usage in settings.render~~ T133942 **Outcome**: [] Comment with results of the analysis [] Ping by email to reading web team for attention and discussion

Duration: 8hrs Let's review the existing JavaScript code before pushing to stable. **Analyze**: [] How does it impact first render time / fully loaded load time [] How long does a hovercard take to load (especially as @nuria points out on slow connections) * ~~Which libraries does it use and which are loaded asynchronously~~ T133947 * ~~Review Icon support to see if it will benefit from ResourceLoaderImageModule~~ T133956 * ~~ext.popups.settings.js would benefit from template usage in settings.render~~ T133942 **Outcome**: [] Comment with results of the analysis [] Ping by email to reading web team for attention and discussion

Duration: 8hrs Let's review the existing JavaScript code before pushing to stable. **Analyze**: [] How does it impact first render time / fully loaded load time [] How long does a hovercard take to load (especially as @nuria points out on slow connections) [] * ~~Which libraries does it use and which are loaded asynchronously~~ T133947 * ~~Review Icon support to see if it will benefit from ResourceLoaderImageModule~~ T133956 * ~~ext.popups.settings.js would benefit from template usage in settings.render~~ T133942 **Outcome**: [] Comment with results of the analysis [] Ping by email to reading web team for attention and discussion