Skip to content

Update Neutralino.js in tools.json #120

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CosmoMyzrailGorynych
Copy link

@CosmoMyzrailGorynych CosmoMyzrailGorynych commented Jan 29, 2025

Removed some now invalid or purely subjective points.

  • Tray icons exist for a loooong time.
  • Not sure what you meant by "custom window chrome". There are chrome and window options which allow you to either create a minimal Chrome window or use the webview library with more window control.
  • App icons:
    • I made a PR some time ago and you can now set Windows executable's icon with cross-building support. Linux and Mac icons are not embedded into executables and usually require work outside of your app itself.
      • Mac will need you to make a Mac bundle, which is just an YourApp.app folder + a couple files. There are build scripts for that from community and the process to make your own bundle is pretty straightforward and googleable.
      • Linux icons has a similar case where executables themselves don't have icons but you create .desktop files for them. Usually these are made by making an installer or a package like flatpak (which is out of Neutralino's and many other XPDA frameworks' scope.)
  • Filesystem access: I don't agree with that much, there are ofc several cases where some sugar can be added, but Node.js doesn't have such sugar per se. Like ensureFolder or outputFile from fs-extra. These are all doable with simple methods and I have a small internal lib with all that sugar in my project, too.
    • The only "hole" I faced when migrating a huge nw.js app is getting access mode on files and folders. (fs.access from node) I polyfilled it by creating a temp text file in the specified folders for W checks and with a direct read for R checks. But, this may change in the future, I should nudge the dev in this direction.
    • I assume some may need to create symlinks, too. There are no methods for that in fs namespace. Right now to do that, you would need to use os.execCommand and do that through CLI.
    • Any other "holes"? If you find some I can bring that point back.
  • "Documentation is pretty light": Can't agree; every aspect, method, configuration option, CLI flag is documented.
  • Crashes: Neutralino works for me and many others; maybe there was an issue at that version of neutralino and they were fixed. Issues I saw before with idle crashes date back to 2023 or earlier. Try again and see and raise an issue if it still happens?

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.

1 participant