felixhao28 / node-systray

A cross-platform systray library for nodejs.
https://www.npmjs.com/package/systray2
MIT License
32 stars 9 forks source link

Add support for dividers #3

Closed Timeraa closed 3 years ago

Timeraa commented 3 years ago

Hey there!

Since the original author doesn't update their module anymore and you seem to be updating it, I've come here in hope to use it for my application PreMiD. It seems to do the trick for my use cases BUT I'd love to see the possibility to use the dividers as the go module seems to support them but this one "lacks?" them.

~ Florian

felixhao28 commented 3 years ago

The separator support is just added (not published yet). Will you try npm i felixhao28/node-systray and add an "SysTray.separator" in the menu array and see if it works for you?

Timeraa commented 3 years ago

👌 Works fine but couldn't install it the proper way, it just had empty folders but locally compiling the files and testing, it works.

Timeraa commented 3 years ago

Also the go library seems to support sub-menus? If so that would be useful to have support for as well

felixhao28 commented 3 years ago

Yeah installing from GitHub won't actually work because I don't commit the compiled js file to the repository.

felixhao28 commented 3 years ago

Sub-menus support has been added to the master branch. You can pull and try again. I only tested on Windows.