Skip to content

Commit 5f52172

Browse files
Use correct input type when collecting email addresses
When filling out a form that collects an email address, I want my email to be validated in the browser so that I don’t find out it was incorrectly formatted when submitting the form. Issues ------ - Closes #73
1 parent a92db88 commit 5f52172

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

Diff for: README.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ end
216216
<%= render partial: "shared/form_errors", locals: { object: form.object } %>
217217
<div>
218218
<%= form.label :email %>
219-
<%= form.text_field :email, required: true %>
219+
<%= form.email_field :email, required: true %>
220220
</div>
221221
<div>
222222
<%= form.label :password %>
@@ -557,7 +557,7 @@ end
557557
<%= form_with url: login_path, scope: :user do |form| %>
558558
<div>
559559
<%= form.label :email %>
560-
<%= form.text_field :email, required: true %>
560+
<%= form.email_field :email, required: true %>
561561
</div>
562562
<div>
563563
<%= form.label :password %>
@@ -992,7 +992,7 @@ end
992992
<%= render partial: "shared/form_errors", locals: { object: form.object } %>
993993
<div>
994994
<%= form.label :email, "Current Email" %>
995-
<%= form.text_field :email, disabled: true %>
995+
<%= form.email_field :email, disabled: true %>
996996
</div>
997997
<div>
998998
<%= form.label :unconfirmed_email, "New Email" %>

Diff for: app/views/sessions/new.html.erb

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<%= form_with url: login_path, scope: :user do |form| %>
22
<div>
33
<%= form.label :email %>
4-
<%= form.text_field :email, required: true %>
4+
<%= form.email_field :email, required: true %>
55
</div>
66
<div>
77
<%= form.label :password %>

Diff for: app/views/users/edit.html.erb

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<%= render partial: "shared/form_errors", locals: { object: form.object } %>
33
<div>
44
<%= form.label :email, "Current Email" %>
5-
<%= form.text_field :email, disabled: true %>
5+
<%= form.email_field :email, disabled: true %>
66
</div>
77
<div>
88
<%= form.label :unconfirmed_email, "New Email" %>

Diff for: app/views/users/new.html.erb

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<%= render partial: "shared/form_errors", locals: { object: form.object } %>
33
<div>
44
<%= form.label :email %>
5-
<%= form.text_field :email, required: true %>
5+
<%= form.email_field :email, required: true %>
66
</div>
77
<div>
88
<%= form.label :password %>

0 commit comments

Comments
 (0)