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

specified icon not being used when generating osx build #519

Closed
symbyte opened this issue Jun 16, 2016 · 5 comments · May be fixed by qcif/data-curator#563
Closed

specified icon not being used when generating osx build #519

symbyte opened this issue Jun 16, 2016 · 5 comments · May be fixed by qcif/data-curator#563

Comments

@symbyte
Copy link
Contributor

symbyte commented Jun 16, 2016

specifying a build to use a specific icon when building like this:

  "build": {
    "osx": {
      "icon": "./images/myIcon.icns"
    }
  },

the icon is only being used in the .dmg installation title bar and for the mounted dmg on the desktop, but the default electron icon is being used everywhere else (app tray icon, etc.). Is there anything that I need to do besides what I have shown here to get this behavior?

@develar
Copy link
Member

develar commented Jun 17, 2016

Do you see warn about icon in the build log? Do you have icon in the build/icon.icns?

@symbyte
Copy link
Contributor Author

symbyte commented Jun 17, 2016

I do not see the warning. I do not have the icon at build/icon.icns because I assumed that specifying the icon field in the build property in package.json overrides that. Does it still need to be there?

@develar
Copy link
Member

develar commented Jun 17, 2016

I do not see the warning.
@symbyte Do you use electron-builder 5.4.x?

I do not have the icon at build/icon.icns because I assumed that specifying the icon field in the build property in package.json overrides that. Does it still need to be there?

https://github.com/electron-userland/electron-builder/wiki/Options#OsXBuildOptions-icon The path to DMG icon Yeach, it is not user-friendly, will be fixed somehow soon.

@symbyte
Copy link
Contributor Author

symbyte commented Jun 17, 2016

D'oh. Okay, placing icon.icns in the right place causes the expected behavior. Thanks for your clarification!

@develar
Copy link
Member

develar commented Jun 17, 2016

Enough. We got enough feedback.

  • deprecate osx
  • introduce dmg. use osx if present and warn that it is deprecated. all appdmg options goes here.
  • introduce mac — all mac platform options goes here. To not break existing behaviour, use osx as base and warn that it is deprecated.
  • allow to override app icon using mac.icon.

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 a pull request may close this issue.

2 participants