Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 7, 2024. It is now read-only.

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.

3 participants