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
Minor ui improvements #1034
Minor ui improvements #1034
Conversation
public async onLogin(passPhrase: string) { | ||
const trimmed = passPhrase ? passPhrase.trim() : passPhrase; | ||
const passPhraseTrimmed = passPhrase.trim(); | ||
|
||
try { | ||
await window.onLogin(trimmed); | ||
await window.onLogin(passPhraseTrimmed); | ||
} catch (error) { | ||
// Increment the error counter and show the button if necessary | ||
this.setState({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not an issue with this PR but is that a good idea to trim the passphrase? I mean if a user uses a password with by any luck a space at the end? It's ok for me but we should double check every time we save the password we trim it also.
Otherwise the user might be able to save a password with a space at the end but will never be able to login with it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. I'll address this in a new PR
MIN_HEIGHT
and other window params to accommodate modal heightsSessionPasswordPrompt
.