Incorporate review feedback

Export app before requiring modules

Remove label/accelerators with role defaults

Add assert for role with app name in label

:art:

Remove lint errors

Doc role defaults

Add spec for default role label/accelerator

Specify focused window to execute call

Add app require

Add execute helper to roles file

:art: Sort roles alphabetically

Add more role defaults

Add default help/window labels

Use default labels and accelerators

Add default label/accelerator to role menu items

Add menu item role defaults

Merge pull request #6184 from electron/read-only-command-id

Make MenuItem.commandId read only

Make commandId a read only property

Add failing spec for writeable command id

Merge pull request #6182 from electron/format-menu-item

Reformat MenuItem class

Add back event param

Move click variable near this.click

:art: Destructure requires

Add spec for submenu type with no submenu

Add spec for invalid menu item type

Remove CoffeeScript class wrapper

Merge pull request #6175 from miniak/content-protection

Add BrowserWindow.prototype.setContentProtection(enable)

Merge pull request #6173 from arifcakiroglu/patch-1

Update README.md

Add BrowserWindow.prototype.setContentProtection(enable)

Update README.md

Merge pull request #6172 from electron/session-setuseragent

Add Session.setUserAgent(userAgent[, acceptLanguages]) API

Update README.md

add `electron-tr` community

Fix usages of global BrowserContext in App::ImportCertificate

docs: ses.setUserAgent(userAgent[, acceptLanguages])

Add ses.getUserAgent()/setUserAgent(...) APIs

Cache the BrowserContext::GetUserAgent

Do not force creating default session in C++

Merge pull request #6171 from electron/maximize/minimize

mac: Fix maximize/unmaximize event emitted before window is maximized

mac: Fix maximize/unmaximize event emitted before window is maximized

Merge pull request #6170 from electron/click-event

Add "event" parameter for "click" handler of MenuItem

spec: Fix failing tests of Menu

docs: Reformat the menu-item.md

Add "event" parameter for "click" handler of MenuItem

Add EventEmitter::CreateEventFromFlags

Bump v1.2.4

Merge pull request #6169 from electron/map-roles-by-command-id

Map roles by command id instead of index

Merge pull request #6168 from electron/forward-delete

Support forward delete menu item accelerator

Merge pull request #6166 from electron/fullscreen-role

Add togglefullscreen menu item role

Add comment about map key/values

Map roles by command id

Add delete accelerator to delete menu

Make NSDeleteFunctionKey to NSDeleteCharacter

Access function in methodInBrowserWindow

Doc togglefullscreen menu item role

:art:

Use togglefullscreen role

Add togglefullscreen role selector

Add togglefullscreen role

Merge pull request #6154 from electron/quit-role

Add quit menu item role

Fix lint errors

Document quit role

Remove CoffeeScript ref variables

Use const/let instead of var

Add File > Exit menu on Windows

Use quit role

Map quit role to terminate selector

Add quit role

Merge pull request #6161 from electron/refresh-menu-bar-color

Update menubar color when theme changes

Update menubar color when theme changes

linux: Disable parent window when showing modal dialogs (#6160)

Merge pull request #6159 from electron/tray-get-bounds

docs: Make tray.md follow docs style guide

docs: Tray.getBounds()

win: Use DIP rect for tray icon's bounds

win: Implement tray.getBounds() API

mac: Add tray.getBounds() API

Update brightray for electron/brightray#228

Add default error handler to remote promises (#6151)

* Add failing spec for unhandled main process exception

* Remove unused return

* Use let/const instead of var

* Add spec for unhandled rejection in renderer process

* Prevent unhandled rejection defaul

* Use once instead of on

* Add default fulfilled/rejection handler to promise

Add systemPreferences.subscribeLocalNotification. (#6150)

Merge pull request #6147 from electron/open-item-in-folder

mac: Convert path to absolute path in ShowItemInFolder

mac: Convert path to absolute path in ShowItemInFolder

Merge pull request #6145 from electron/app-indicator-id

Use fixed ID for app indicators

Use fixed ID for app indicators

Merge pull request #6144 from electron/crash-release-renderer

Avoid calling Environment::GetCurrent for unjected contexts

Avoid calling Environment::GetCurrent for unjected contexts

It seems to cause crashes when it is called for contexts that do not
have node integrations.

Merge pull request #6140 from electron/parent

Add support for child windows

docs: modal window is greatly simplified

Only define enable/disable API for views

macOS: Use sheet window as modal window

Move disable counter to NativeWindow from api::Window

docs: Parent and child windows

linux: Make modal dialog show more smoothly

linux: Implement win.disable/enable/isEnabled() API

Do not emit "unresponsive" when there is modal dialog

linux: Implement win.setParentWindow(parent)

win: Implement win.disable/enable/isEnabled() API

spec: Avoid calling setParentWindow for unrelated tests

Add "parent" option for BrowserWindow

views: Initial implementation of win.setParentWindow(parent)

But it doesn't work, aparrently Chromium doesn't support changing
parent window dynamically on desktop.