Skip to content

feat: add more info in setWindowOpenHandler details#29276

Closed
VerteDinde wants to merge 1 commit into13-x-yfrom
13-x-y-backport-window-open-opts
Closed

feat: add more info in setWindowOpenHandler details#29276
VerteDinde wants to merge 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
@VerteDinde VerteDinde requested a review from nornagon May 21, 2021 16:50
@VerteDinde VerteDinde requested review from a team as code owners May 21, 2021 16:50
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label May 21, 2021
@trop trop bot added backport This is a backport PR backport/requested 🗳 master 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 VerteDinde changed the base branch from master to 13-x-y May 21, 2021 16:51
@trop trop bot added 13-x-y semver/minor backwards-compatible functionality and removed semver/minor backwards-compatible functionality labels May 21, 2021
@VerteDinde VerteDinde closed this May 21, 2021
@VerteDinde VerteDinde deleted the 13-x-y-backport-window-open-opts branch May 21, 2021 16:52
@VerteDinde VerteDinde restored the 13-x-y-backport-window-open-opts branch May 21, 2021 16:53
@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.

2 participants