Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Add --browserwindow-options to completely expose Electron options #835

Merged
merged 5 commits into from
Aug 22, 2019
Merged

Add --browserwindow-options to completely expose Electron options #835

merged 5 commits into from
Aug 22, 2019

Conversation

FabulousCupcake
Copy link
Contributor

@FabulousCupcake FabulousCupcake commented Aug 21, 2019

This PR adds a new option to the CLI (--browserwindow-options), taking a stringified JSON object as input.
It will be inserted directly into the options when BrowserConfig is initialised in /app/src/components/mainWindow/mainWindow.js.

This allows total configurability of the electron BrowserWindow configuration via nativefier.

An example use case is added to the documentation, which modifies the default font family in the browser.

References

@ronjouch
Copy link
Contributor

@FabulousCupcake thanks! Merging.

@ronjouch ronjouch merged commit d0a0614 into nativefier:master Aug 22, 2019
@FabulousCupcake FabulousCupcake deleted the feature/passthrough-options branch August 22, 2019 19:42
Adam777Z pushed a commit to Adam777Z/nativefier that referenced this pull request Nov 9, 2022
…ativefier#835)

This adds a new flag `--browserwindow-options`, taking a stringified JSON object as input.  
It will be inserted directly into the options when BrowserConfig is initialized.

This allows total configurability of the electron BrowserWindow configuration via nativefier.

An example use case is added to the documentation, which modifies the default font family in the browser.

#### References

- https://github.com/electron/electron/blob/master/docs/api/browser-window.md#new-browserwindowoptions
  - See the following for electron v3.1.7
  - https://github.com/electron/electron/blob/v3.1.7/docs/api/browser-window.md#new-browserwindowoptions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants