From d0064716a4ee365e33eb048ee95ecc78a8ea1325 Mon Sep 17 00:00:00 2001 From: Milan Burda Date: Sun, 12 May 2019 18:13:08 +0200 Subject: [PATCH] fix: 'page-title-updated' event forwarding + documentation --- docs/api/browser-window.md | 2 ++ docs/api/web-contents.md | 11 +++++++++++ lib/browser/api/browser-window.js | 4 ++-- 3 files changed, 15 insertions(+), 2 deletions(-) diff --git a/docs/api/browser-window.md b/docs/api/browser-window.md index 42880cba0af3b..91707ba853715 100644 --- a/docs/api/browser-window.md +++ b/docs/api/browser-window.md @@ -406,9 +406,11 @@ Returns: * `event` Event * `title` String +* `explicitSet` Boolean Emitted when the document changed its title, calling `event.preventDefault()` will prevent the native window's title from changing. +`explicitSet` is false when title is synthesized from file url. #### Event: 'close' diff --git a/docs/api/web-contents.md b/docs/api/web-contents.md index fd96d6d53eb2a..7df2b6b282356 100644 --- a/docs/api/web-contents.md +++ b/docs/api/web-contents.md @@ -101,6 +101,17 @@ Returns: Emitted when the document in the given frame is loaded. +#### Event: 'page-title-updated' + +Returns: + +* `event` Event +* `title` String +* `explicitSet` Boolean + +Fired when page title is set during navigation. `explicitSet` is false when +title is synthesized from file url. + #### Event: 'page-favicon-updated' Returns: diff --git a/lib/browser/api/browser-window.js b/lib/browser/api/browser-window.js index 3694acd119c99..a35747afe69db 100644 --- a/lib/browser/api/browser-window.js +++ b/lib/browser/api/browser-window.js @@ -39,9 +39,9 @@ BrowserWindow.prototype._init = function () { }) // Change window title to page title. - this.webContents.on('page-title-updated', (event, title) => { + this.webContents.on('page-title-updated', (event, title, ...args) => { // Route the event to BrowserWindow. - this.emit('page-title-updated', event, title) + this.emit('page-title-updated', event, title, ...args) if (!this.isDestroyed() && !event.defaultPrevented) this.setTitle(title) })