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

Hide main window and add offline mode link to login dialog #286

Closed
sssoleileraaa opened this issue Mar 27, 2019 · 2 comments · Fixed by #313
Closed

Hide main window and add offline mode link to login dialog #286

sssoleileraaa opened this issue Mar 27, 2019 · 2 comments · Fixed by #313

Comments

@sssoleileraaa
Copy link
Contributor

sssoleileraaa commented Mar 27, 2019

Description

@ninavizz
Copy link
Member

Text edits:

  • Sign In to use this newsroom's SecureDrop Workstation
    • ...since we don't yet capture newsroom names.
  • Passphrase
    • ...since that's existing SD nomenclature.
  • 2-Factor Number
    • ...since the designer was just being lazy and accidentally left 2FA in the above wireframe.
    • Note to @eloquence and @redshiftzero: I've flipped-back to 2-Factor, from 2-Step, after poking around some more and looking at other naming conventions against what their patterns are.
      • "2-Step" appears to be used when it's a text-message or some other service-side action a user needs to respond to, whereas "2-Factor" appears to be used when an RSA-token kind of application is used for users to auto-generate the disposable numbers.
      • Happy to discuss in more depth.
      • My efforts to fish around for conclusive research evidence garnered none. :(

@sssoleileraaa
Copy link
Contributor Author

Styling for this issue has been moved to here: #311
This issue focuses on no longer showing the main window when the application launches until the user logs in or clicks on the offline mode link in the login dialog.

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.

2 participants