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

#25 Added required field to gmail and passwords and make them mandatory #27

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions dist/register.html
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@
type="text"
id="FirstName"
name="first_name"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm" required
/>
</div>

Expand All @@ -79,20 +79,20 @@
type="text"
id="LastName"
name="last_name"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm" required
/>
</div>

<div class="col-span-6">
<label for="Email" class="block text-sm font-medium text-gray-700">
Email
Email<span class="mandatory">*</span>
</label>

<input
type="email"
id="Email"
name="email"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm" required
/>
</div>

Expand All @@ -101,14 +101,14 @@
for="Password"
class="block text-sm font-medium text-gray-700"
>
Password
Password<span class="mandatory">*</span>
</label>

<input
type="password"
id="Password"
name="password"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm" required
/>
</div>

Expand All @@ -117,14 +117,14 @@
for="PasswordConfirmation"
class="block text-sm font-medium text-gray-700"
>
Password Confirmation
Password Confirmation<span class="mandatory">*</span>
</label>

<input
type="password"
id="PasswordConfirmation"
name="password_confirmation"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm"
class="mt-1 w-full rounded-md border-gray-200 bg-white text-sm text-gray-700 shadow-sm" required
/>
</div>

Expand All @@ -134,11 +134,11 @@
type="checkbox"
id="MarketingAccept"
name="marketing_accept"
class="h-5 w-5 rounded-md border-gray-200 bg-white shadow-sm"
class="h-5 w-5 rounded-md border-gray-200 bg-white shadow-sm" required
/>

<span class="text-sm text-gray-700">
I want to receive emails about events, module updates and job guidance.
<span class="mandatory">*</span> I want to receive emails about events, module updates and job guidance.
</span>
</label>
</div>
Expand Down
7 changes: 7 additions & 0 deletions dist/style.css
Original file line number Diff line number Diff line change
Expand Up @@ -404,6 +404,13 @@ Make sure disabled buttons don't get the pointer cursor.
cursor: default;
}

/* Make mandatory textfields */
.mandatory {
color: red;
margin-left: 5px;
}


/*
1. Make replaced elements `display: block` by default. (https://github.com/mozdevs/cssremedy/issues/14)
2. Add `vertical-align: middle` to align replaced elements more sensibly by default. (https://github.com/jensimmons/cssremedy/issues/14#issuecomment-634934210)
Expand Down