Last modified: 2013-08-03 22:51:31 UTC
Simply test it: During the whole upload process 2 intervalls *per file* are not cleared: var se = window.setInterval; window.setInterval = function (fn, t) { return se(function() { console.log(fn, t); fn(); }, t) }; If you upload 50 files, this means 100 events fire (all the time) within 500ms, executing a function that is computing some numbers from the DOM (object position) and setting CSS. Not good for old machines and slow browsers. Culprit: /extensions/UploadWizard/resources/mw.UploadWizardUploadInterface.js moveFileInputToCover: function( selector ) { //... this.moveFileInputInterval = window.setInterval(function() { update(); }, 500); }
Change 75122 had a related patch set uploaded by Rillke: Improved file-control positioning https://gerrit.wikimedia.org/r/75122
Change 75122 abandoned by Rillke: Improved file-control positioning Reason: Too much work to do. No time fixing boring white-space. https://gerrit.wikimedia.org/r/75122
Change 75122 restored by Alex Monk: Improved file-control positioning Reason: Don't abandon changes for such silly reasons. https://gerrit.wikimedia.org/r/75122
Created attachment 13056 [details] Screenshot showing Firebug, UploadWizard and Mozilla Firefox. CSS has been tweaked to make the input visible.
Change 75122 merged by jenkins-bot: Improved file-control positioning https://gerrit.wikimedia.org/r/75122