Bug 1595064 Comment 2 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Hmm, not sure if this is a bug or intended. We have separate handling for PDF/document zooming than browser zooming; will try to find a searchfox link if that'd be helpful. NI'ing asa and Jamie to decide if this is something worth modifying in other (ongoing) zoom changes.
Revision 1 by
on 
Hmm, not sure if this is a bug or intended. We have separate handling for PDF/document zooming vs. browser zooming; will try to find a searchfox link if that'd be helpful. NI'ing asa and Jamie to decide if this is something worth modifying in other (ongoing) zoom changes.
Revision 2 by
on 
Hmm, not sure if this is a bug or intended. We have separate handling for PDF/document zooming vs. browser zooming; will try to find a searchfox link if that'd be helpful. NI'ing asa and Jamie to decide if this is something worth modifying in other (ongoing) zoom changes.

EDIT: https://searchfox.org/mozilla-central/rev/e7c61f4a68b974d5fecd216dc7407b631a24eb8f/browser/base/content/browser-fullZoom.js#275
if you follow that `applyToPref` code, the handling for an undefined `aValue` argument is to reset zoom to the default (100%) browser level, leaving zooming on a PDF viewer page to the PDF viewer itself.

Back to Bug 1595064 Comment 2