Skip to content

feat: add more info in setWindowOpenHandler details (#28518)#29277

Merged
zcbenz merged 1 commit into13-x-yfrom
13-x-y-backport-window-open-opts
May 24, 2021
Merged

feat: add more info in setWindowOpenHandler details (#28518)#29277
zcbenz merged 1 commit into13-x-yfrom
13-x-y-backport-window-open-opts

Conversation

@VerteDinde
Copy link
Copy Markdown
Member

Manual backport of #28518

See that PR for details.

Notes: Added disposition, referrer and postBody to the details object passed to the window open handler registered with setWindowOpenHandler.

* fix: invoke the window open handler for _blank links

* feat: add disposition to setWindowOpenHandler details

* fix: pass postData to new-window event

* postData can be heterogeneous

* fix type of postBody

* fix type of UploadFile and UploadRawData to be discriminated unions

* exclude the empty string from additionalFeatures

* add a test

* add postBody and referrer to setWindowOpenHandler args

* appease typescript

* Update api-browser-window-spec.ts

* update snapshots
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label May 21, 2021
@trop trop bot added 13-x-y backport This is a backport PR backport/requested 🗳 semver/minor backwards-compatible functionality labels May 21, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label May 21, 2021
@VerteDinde
Copy link
Copy Markdown
Member Author

Fairly certain the build-mac failures are related to a current CircleCI outage, but will confirm before merging

@zcbenz zcbenz merged commit c69d0ee into 13-x-y May 24, 2021
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented May 24, 2021

Release Notes Persisted

Added disposition, referrer and postBody to the details object passed to the window open handler registered with setWindowOpenHandler.

@VerteDinde VerteDinde deleted the 13-x-y-backport-window-open-opts branch May 24, 2021 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

13-x-y backport This is a backport PR semver/minor backwards-compatible functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants