Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

electron: add missing fullscreen option #3192

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

khassel
Copy link
Collaborator

@khassel khassel commented Sep 13, 2023

follow up for #3187

@bugsounet can you please confirm that this now works for you?

@bugsounet
Copy link
Contributor

works but ...

webPreferences: {
        fullscreen: false
      }

not work

@khassel
Copy link
Collaborator Author

khassel commented Sep 13, 2023

  electronOptions: {
    width: 800,
    height: 600,
    fullscreen: false,
  },

works on my side

@bugsounet
Copy link
Contributor

tested with your electronOptions: {} and don't work for me :/

@khassel
Copy link
Collaborator Author

khassel commented Sep 13, 2023

strange, I used this in config.js:

/* MagicMirror       Config Sample
 *
 * By Michael Teeuw https://michaelteeuw.nl
 * MIT Licensed.
 *
 * For more information on how you can configure this file
 * see https://docs.magicmirror.builders/configuration/introduction.html
 * and https://docs.magicmirror.builders/modules/configuration.html
 */
let config = {

  electronOptions: {
    width: 800,
    height: 600,
    fullscreen: false,
  },

        address: "0.0.0.0",     // Address to listen on, can be:
                                                        // - "localhost", "127.0.0.1", "::1" to listen on loopback interface
                                                        // - another specific IPv4/6 to listen on a specific interface
                                                        // - "0.0.0.0", "::" to listen on any interface
                                                        // Default, when address config is left out or empty, is "localhost"
        port: 8080,
...

and it is 800x600 with taskbar, tested on a pi 4.

Did you test on a pi?

@bugsounet
Copy link
Contributor

ah shit, it's me ... I have set it in webPreferences of electronOptions :/

So it's works :)

@khassel khassel marked this pull request as ready for review September 13, 2023 22:07
@bugsounet
Copy link
Contributor

(we open the bets? e2e will work or not ??)

@khassel
Copy link
Collaborator Author

khassel commented Sep 13, 2023

(we open the bets? e2e will work or not ??)

I hate the e2e tests ...

@bugsounet
Copy link
Contributor

it will works PASS e2e tests/e2e/modules_position_spec.js

@khassel khassel requested a review from rejas September 13, 2023 22:13
@rejas rejas merged commit 7127979 into MagicMirrorOrg:develop Sep 14, 2023
@khassel khassel deleted the electron-fullscreen branch September 16, 2023 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants