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

cmd/trayscale: clean up data flow #19

Open
3 tasks
DeedleFake opened this issue Sep 15, 2022 · 0 comments · May be fixed by #89
Open
3 tasks

cmd/trayscale: clean up data flow #19

DeedleFake opened this issue Sep 15, 2022 · 0 comments · May be fixed by #89
Assignees

Comments

@DeedleFake
Copy link
Owner

Thanks to the Prefs() rework from #18, data flow from Tailscale to the UI is a tad strange. It mostly comes from the status, but a few items are from prefs instead, and those are being pulled on demand during the update. This is rather odd, so here's the current plan to fix it:

  • Move status fetching into the GTK event and rework polling loop to just be a simple timer that fires the custom GTK update event.
  • Poll prefs at the beginning of the event.
  • Manually update switches directly via return values from methods.
@DeedleFake DeedleFake self-assigned this Sep 15, 2022
@DeedleFake DeedleFake changed the title cmd/trayscale: fix data flow cmd/trayscale: clean up data flow Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant