You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the primary use-case for the zoomDisabledTimeout functionality being removed in PR #17724, I wonder if we could get rid of the rest of this code now?
The remaining code was added in PR #10351 to work-around bug 1503412, but without access to a Mac I cannot tell if we still need that and if so whether it's possible to refactor it to remove the setTimeout-based hack?
Snuffleupagus
changed the title
[web/app.js] Remove the remaing zoomDisabledTimeout usage?
[web/app.js] Remove the remaining zoomDisabledTimeout usage?
Feb 26, 2024
With the primary use-case for the
zoomDisabledTimeout
functionality being removed in PR #17724, I wonder if we could get rid of the rest of this code now?The remaining code was added in PR #10351 to work-around bug 1503412, but without access to a Mac I cannot tell if we still need that and if so whether it's possible to refactor it to remove the
setTimeout
-based hack?/cc @calixteman The following patch simply removes the affected code, would you be able to test it on Mac?
master...Snuffleupagus:pdf.js:rm-setZoomDisabledTimeout
The text was updated successfully, but these errors were encountered: