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

Fix Safari support for sizes/srcset attributes/properties #16961

Merged
merged 2 commits into from
Jul 13, 2022

Conversation

foolip
Copy link
Contributor

@foolip foolip commented Jul 13, 2022

The <picture> element was in WebKit trunk version 602.1.14:
WebKit/WebKit@ee1f266
https://github.com/WebKit/WebKit/blob/ee1f2663e07eb21f6ea8e0b0d01fce127d66e9a7/Source/WebCore/Configurations/Version.xcconfig

The reflected properties were in WebKit trunk version 603.1.6:
WebKit/WebKit@96c370b
https://github.com/WebKit/WebKit/blob/96c370b3601a5427ba1a4dd2ef78657d59729721/Source/WebCore/Configurations/Version.xcconfig

Add notes to reflect the discrepancy, in case we try to merge attributes
and reflected properties in the future.

@foolip foolip requested a review from queengooborg July 13, 2022 08:51
@github-actions github-actions bot added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML labels Jul 13, 2022
Copy link
Contributor

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

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

Got some problems with the notes, but otherwise looking good!

api/HTMLSourceElement.json Outdated Show resolved Hide resolved
api/HTMLSourceElement.json Outdated Show resolved Hide resolved
Co-authored-by: Queen Vinyl Da.i'gyu-Kazotetsu <[email protected]>
@foolip foolip requested a review from queengooborg July 13, 2022 10:45
@queengooborg queengooborg merged commit 7dacd85 into mdn:main Jul 13, 2022
@foolip foolip deleted the safari-sizes-srcset branch July 13, 2022 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants