-
Notifications
You must be signed in to change notification settings - Fork 267
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
Don't show Settings window on App Start #3961
Comments
Agreed. But there is a problem that Windows hides our tray icon by default, and a tray icon may not be an obvious enough launch notification on the first time. So we can try forcing the tray icon to show, at least while Firezone is signed out: https://superuser.com/questions/1753209/ And we could use the generation of the device ID (firezone-id) as a cue to do a little "first-time install" thing, maybe have a notification saying, "Firezone is in the tray, sign in to access resources" At the most extreme we could show a notification every time Firezone starts signed-out saying "Sign in to Firezone" |
I like the E.g. "Welcome to Firezone. Use the taskbar icon to sign in." I can whip up a graphic / view |
Ah but I guess on Linux it'd be in the upper-right most likely. Instead of the arrow we could just show a zoomed in screenshot of the icon running in the Taskbar / Menubar |
Oh, does Tauri support UI notifications? Maybe we could show that on first launch instead? |
Either or, I don't feel strongly. The macOS / iOS / Android apps shows a welcome screen with the "Grant VPN permission" button. On Tauri it could be similar, minus the "Grant" buton |
Agreed on standup to add a third window "Welcome" which can't be explicitly shown but is shown after first install and then not shown once the user dismisses it (for some bikeshed-able definition of "dismiss") I'm not sure if we can somehow detect an "Install ID" for this case:
|
Maybe it can be assumed the user already knows where to find Firezone in this case. I think it's mostly to help users who have never seen Firezone at all |
Closes #3961 No tests yet, might be tricky to test since it's all I/O. I cued it off the device ID being generated, so it will have a minor merge conflict with #3920 ```[tasklist] ### Before merging - [ ] UI polish, or disable the welcome screen temporarily ``` <img width="664" alt="image" src="https://github.com/firezone/firezone/assets/13400041/d5def59c-b075-4135-91e5-85f9f9212fa5"> --------- Co-authored-by: Jamil Bou Kheir <jamilbk@users.noreply.github.com>
No description provided.
The text was updated successfully, but these errors were encountered: