Skip to content
This repository has been archived by the owner on Jul 7, 2024. It is now read-only.
/ Orion Public archive

Add option to disable the tray icon #108

Merged
merged 2 commits into from
May 16, 2018

Conversation

kernelwhisperer
Copy link
Contributor

What changed?

  • Added a one-time notification when you first close the app to let the user know it will run the background.
  • Added setting to enable/disable the tray icon

app/index.js Outdated

const systemTrayNotification = Settings.getSync('systemTrayNotification')

if (!systemTrayNotification) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will trigger the message every time the settings is false, try with a more specific setup:
systemTrayNotification === undefined

* Repository Panel
*/
@observer
class AppPanel extends React.Component {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need a new panel for one option? Can we add it to the Daemon settings?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added it to the connectivity panel, we don't have a daemon panel anymore

Copy link
Member

@koalalorenzo koalalorenzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🔥 👍

@kernelwhisperer kernelwhisperer merged commit 25a90f5 into master May 16, 2018
@kernelwhisperer kernelwhisperer deleted the feature/121-tray-icon-setting branch May 16, 2018 17:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants