Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compatible with ResizeObserver borderBoxSize in legacy safari #16311

Merged

Conversation

linxianxi
Copy link
Contributor

@linxianxi linxianxi commented Apr 18, 2023

Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing that this is specifically for Safari-compatibility[1], since the commit message doesn't actually say which browser(s) are affected?
Hence, when updating the patch, please squash the commits and also improve the commit message.


[1] Based on https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support and https://developer.mozilla.org/en-US/docs/Web/API/ResizeObserverEntry/borderBoxSize#browser_compatibility

web/pdf_viewer.js Outdated Show resolved Hide resolved
@linxianxi linxianxi force-pushed the fix-viewer-browser-compatibility branch from 004f7cf to 19eb845 Compare April 18, 2023 08:46
@linxianxi linxianxi changed the title Compatible with borderBoxSize in older browsers Compatible with ResizeObserver borderBoxSize in safari Apr 18, 2023
@linxianxi linxianxi changed the title Compatible with ResizeObserver borderBoxSize in safari Compatible with ResizeObserver borderBoxSize in legacy safari Apr 18, 2023
@linxianxi linxianxi force-pushed the fix-viewer-browser-compatibility branch 3 times, most recently from 2106eec to 2ac1b2f Compare April 18, 2023 09:19
@linxianxi linxianxi force-pushed the fix-viewer-browser-compatibility branch from f95b464 to 6ca702d Compare April 18, 2023 09:40
@Snuffleupagus
Copy link
Collaborator

/botio-linux preview

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://54.241.84.105:8877/6417634311a9ab9/output.txt

@pdfjsbot
Copy link

From: Bot.io (Linux m4)


Success

Full output at http://54.241.84.105:8877/6417634311a9ab9/output.txt

Total script time: 1.29 mins

Published

Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me, thank you!

@Snuffleupagus Snuffleupagus merged commit 88e2d04 into mozilla:master Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants