-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Upstream shareSheetUsesModalPresentationForWebView #14828
Upstream shareSheetUsesModalPresentationForWebView #14828
Conversation
EWS run on previous version of this PR (hash 5eb02bb) |
will get the header too |
5eb02bb
to
9bbd677
Compare
EWS run on current version of this PR (hash 9bbd677) |
@@ -333,7 +329,7 @@ - (void)presentWithShareDataArray:(NSArray *)sharingItems inRect:(std::optional< | |||
}]; | |||
|
|||
#if PLATFORM(VISION) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't actually need this conditional right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though it will probably cause issues with current EWS if we don't replace with an SDK version check... so never mind.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed :)
https://bugs.webkit.org/show_bug.cgi?id=257913 rdar://110548897 Reviewed by Aditya Keerthi. * Source/WebKit/UIProcess/Cocoa/WKShareSheet.mm: (-[WKShareSheet presentWithShareDataArray:inRect:]): Canonical link: https://commits.webkit.org/265040@main
9bbd677
to
cb63b1b
Compare
Committed 265040@main (cb63b1b): https://commits.webkit.org/265040@main Reviewed commits have been landed. Closing PR #14828 and removing active labels. |
cb63b1b
9bbd677
π wincairoπ§ͺ wpe-wk2π§ͺ ios-wk2π§ͺ api-macπ§ͺ ios-wk2-wptπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk2π§ͺ api-gtkπ tvπ tv-simπ watchπ watch-sim