Skip to content

Commit 32c4bbe

Browse files
committed
Improve visuals of login form fields
This change adds moves avway from borderless + placeholder inputs to larget outlined with label input fields for both email and password input fields on the login form to make the overall appearance of the text fields a bit larger.
1 parent 1ec58d8 commit 32c4bbe

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

identifier/src/containers/Login/Login.jsx

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,11 @@ const styles = theme => ({
4444
message: {
4545
marginTop: theme.spacing(2),
4646
marginBottom: theme.spacing(2)
47-
}
47+
},
48+
usernameInputField: {
49+
marginTop: theme.spacing(1),
50+
marginBottom: theme.spacing(1.5),
51+
},
4852
});
4953

5054
function Login(props) {
@@ -113,11 +117,10 @@ function Login(props) {
113117

114118
<form action="" onSubmit={(event) => this.logon(event)}>
115119
<TextField
116-
placeholder={usernamePlaceHolder}
120+
label={usernamePlaceHolder}
117121
error={!!errors.username}
118122
helperText={<ErrorMessage error={errors.username} values={{what: usernamePlaceHolder}}></ErrorMessage>}
119123
fullWidth
120-
margin="dense"
121124
autoFocus
122125
inputProps={{
123126
autoCapitalize: 'off',
@@ -126,16 +129,18 @@ function Login(props) {
126129
value={username}
127130
onChange={handleChange('username')}
128131
autoComplete="kopano-account username"
132+
variant="outlined"
133+
className={classes.usernameInputField}
129134
/>
130135
<TextField
131136
type="password"
132-
placeholder={t("konnect.login.passwordField.label", "Password")}
137+
label={t("konnect.login.passwordField.label", "Password")}
133138
error={!!errors.password}
134139
helperText={<ErrorMessage error={errors.password}></ErrorMessage>}
135140
fullWidth
136-
margin="dense"
137141
onChange={handleChange('password')}
138142
autoComplete="kopano-account current-password"
143+
variant="outlined"
139144
/>
140145
<DialogActions>
141146
<div className={classes.wrapper}>

0 commit comments

Comments
 (0)