Skip to content
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

Disabling "Watch for Changes" not respected #538

Closed
tristanbrown opened this issue Dec 15, 2019 · 2 comments
Closed

Disabling "Watch for Changes" not respected #538

tristanbrown opened this issue Dec 15, 2019 · 2 comments

Comments

@tristanbrown
Copy link

SyncTrayzor folder-watching is on by default, until it is manually disabled from the File>Settings menu. Disabling "Watch for Changes" in the "Edit Folder" dialogue only disables Syncthing folder-watching, and then SyncTrayzor folder-watching takes over. To disable ALL folder-watching, you additionally need to disable SyncTrayzor folder-watching in the File>Settings menu.

This behavior is very unintuitive, and not indicated in any way by the GUI. What should happen is if I disable "Watch for Changes" in the "Edit Folder" dialogue, ALL folder watching should then be disabled. I'm not sure what the use-case is for SyncTrayzor folder watching instead of Syncthing folder-watching, but it shouldn't be enabled by default if the "Watch for Changes" check box on each folder doesn't control it. Also the "eyeball" watched/not-watched" graphic should refer to all forms of watching, not just Syncthing watching.

@canton7
Copy link
Owner

canton7 commented Dec 15, 2019

SyncTrayzor's folder watching predates Syncthing's by a long way, and disables itself when Syncthing's is enabled to allow a smooth transition from old (SyncTrayzor's) to new (Syncthing's).

Given that Syncthing's folder watching has been in place for a while now, it's probably time to remove SyncTrayzor's for good.

Syncthing's UI can't of course reflect any of SyncTrayzor's settings, because it has no knowledge of SyncTrayzor.

@canton7
Copy link
Owner

canton7 commented Jan 12, 2020

The previous logic was that if any folders had SyncTrayzor's watching enabled, then new folders would too (fine). However it also assumed that watching should be enabled if there were no previous folders, and a folder was added.

Removed that last bit, which should solve your issue.

Stripping out watching altogether is a bigger task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants