-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
Set product name, file description copyright on Windows #30
Comments
Cross-referencing those issues: |
I found this wrapper to resourcehacker, which is probably the most robust solution for modifying compilied Windows executables https://github.com/felicienfrancois/node-resourcehacker We may be able to swap out rcedit for this instead. |
@jden +1 for https://github.com/felicienfrancois/node-resourcehacker sounds like a good alternative. is anybody working on that..?? |
@Rameshv after looking at that repo again, it doesn't appear to be a properly licensed redistribution of resourcehacker... indeed, it's not published on npm. We might be able to make a version of that which downloaded the resourcehacker exe via an npm postinstall script from the original distribution point |
@jden yep thats my concern too. I made a quick fix by implementing the npm postinstall hook asyou suggested. You can see the PR here felicienfrancois/node-resourcehacker#3. Waiting for the response from the author on this. Let me know if this works. |
rcedit works pretty well and can do that (I use grunt-rcedit on my electron app).
To get all the possible keys, you can open the exe with a gui program like resourcehacker or winresourcer PS: I found this on the grunt-rcedit doc |
ah cool, thanks @felicienfrancois . yep rcedit works just fine with the version string info.. thanks for the hint.. it saved my day |
We can now change the version info in resources successfully, but this does not affect the name that appears when you right-click the app in the task bar. It still shows up as "Electron" there even if you change all the resource names. |
This is true. Any ideas regarding @alexwarren 's problem? Maybe this one should be a new issue? |
@alexwarren @s-a It means that you have to set the title in your code when creating a BrowserWindow Same for the icon, you have to set the runtime icon (ie window titlebar and runtime taskbar icon) when creating a BrowserWindow |
@felicienfrancois this is not what we mean. BrowserWindow setup is the same as when I set |
@s-a ok sorry. It seems the name in the taskbar depends if it have been pined and how:
It may also depends on the OS. Windows 10 has changed things. there is an open issue regarding pining on windows 10 electron/electron#1954 |
Closing since #63 was merged. |
Currently the .exe is branded as "Electron" and that shows up when right-clicking the app in the task bar, the task manager and several other key places.
This should be implemented in electron-packager as soon as the issue is resolved in rcedit and node-rcedit, because they currently cannot change those resources (CompanyName, FileDescription, LegalCopyright, ProductName).
I'll keep you posted on this.
The text was updated successfully, but these errors were encountered: