search

jonhoo / buzz

A simple system tray application for notifying about unseen e-mail
Apache License 2.0
138 stars 12 forks source link

Make tray icon optional #21

Closed robinkrahl closed 3 years ago

robinkrahl commented 3 years ago

I know that buzz is advertised as a tray application but I’d like to use it only with notifications and without the system tray icon. Would you accept a PR that makes the tray icon and the systray dependency optional (though enabled by default)?

jonhoo commented 3 years ago

Yes, that seems totally reasonable! Most system trays include the ability to hide individual icons, but it seems reasonable to have the feature built-in too!

robinkrahl commented 3 years ago

Nice! A feature flag has the advantages that it would get rid of most the native dependencies and that it makes libappindicator3 optional which will be removed in the next stable Debian release. I’ll prepare a PR soon.