diff --git a/projects/FrontendMentor/newbie/base-apparel-coming-soon/README.md b/projects/FrontendMentor/newbie/base-apparel-coming-soon/README.md
index a27e461..5962feb 100644
--- a/projects/FrontendMentor/newbie/base-apparel-coming-soon/README.md
+++ b/projects/FrontendMentor/newbie/base-apparel-coming-soon/README.md
@@ -86,17 +86,13 @@ Use this section to outline areas that you want to continue focusing on in futur
## Author
-- Website - [Add your name here](https://www.your-site.com)
-- Frontend Mentor - [@yourusername](https://www.frontendmentor.io/profile/yourusername)
-- Twitter - [@yourusername](https://www.twitter.com/yourusername)
-
-**Note: Delete this note and add/remove/edit lines above based on what links you'd like to share.**
+- Website - [Robert McGovern Blog](https://tarasis.net)
+- Frontend Mentor - [@tarasis](https://www.frontendmentor.io/profile/tarasis)
+- Twitter - [@tarasis](https://www.twitter.com/tarasis)
## Acknowledgments
-This is where you can give a hat tip to anyone who helped you out on this project. Perhaps you worked in a team or got some inspiration from someone else's solution. This is the perfect place to give them some credit.
-
-**Note: Delete this note and edit this section's content as necessary. If you completed this challenge by yourself, feel free to delete this section entirely.**
+My thanks to Julio Cinquina on the FEM Slack for correcting my misunderstanding of the `:has` pseudo-class.
Initial HTML
```html
diff --git a/projects/FrontendMentor/newbie/base-apparel-coming-soon/css/style.css b/projects/FrontendMentor/newbie/base-apparel-coming-soon/css/style.css
index abd250b..1538b7d 100644
--- a/projects/FrontendMentor/newbie/base-apparel-coming-soon/css/style.css
+++ b/projects/FrontendMentor/newbie/base-apparel-coming-soon/css/style.css
@@ -35,7 +35,7 @@
--gradient-pink-hover: linear-gradient(
135deg,
hsl(0, 80%, 86%) 0%,
- hsl(0, 76%, 80%) 100%
+ hsl(0, 76%, 90%) 100%
);
/* Typography */
@@ -63,6 +63,10 @@
--fw-error: var(--fw-400);
--col-error: var(--col-primary-soft-red);
/* Sizing */
+
+ --border-radius: 1.75rem;
+
+ --adjustment-left-field-and-error: 1.5rem;
}
body {
@@ -79,7 +83,7 @@ main {
}
.header {
- margin: 30px 2rem 2rem 2rem;
+ margin: 1.875rem 2rem 2rem 2rem;
grid-area: "logo";
}
@@ -108,11 +112,11 @@ main {
font-size: var(--fs-body);
line-height: var(--lh-body);
+ font-weight: var(--fw-body);
color: var(--col-body);
}
.hero-picture {
- /* display: block; */
grid-area: "hero-image";
justify-self: end;
margin: 0 auto;
@@ -135,17 +139,19 @@ main {
}
.email-form__input {
border: 1px solid var(--col-field);
- /* border: none; */
width: 40ch;
font-size: var(--fs-field);
line-height: var(--lh-field);
- padding: 10px 0 10px 24px;
- border-radius: 28px;
+ font-weight: var(--fw-field);
+ padding: 0.625rem 0 0.625rem var(--adjustment-left-field-and-error);
+ border-radius: var(--border-radius);
background: inherit;
color: var(--col-neutral-dark-grayish-red);
- /* background-color: var(--col-primary-desaturated-red); */
- /* margin-left: 24px; */
+}
+
+.error-border {
+ border: 1px solid var(--col-error);
}
::placeholder {
@@ -155,33 +161,38 @@ main {
.email-form__button-submit {
position: absolute;
- padding: 13px 27.1px 13px 28px;
- border-radius: 28px;
- box-shadow: 0px 15px 20px rgba(198, 110, 110, 0.247569);
+ padding: 0.8125rem 1.6938rem 0.8125rem 1.75rem;
+ border-radius: var(--border-radius);
+ box-shadow: 0px 0.9375rem 1.25rem rgba(198, 110, 110, 0.247569);
background: var(--gradient-pink);
- right: 0px;
+ right: 0;
border: none;
cursor: pointer;
}
-.email-form__button-submit:has(:hover, :focus) {
- background: red;
+.email-form__button-submit:where(:hover, :focus) {
+ background: var(--gradient-pink-hover);
}
.email-form__error-message {
color: var(--col-error);
font-size: var(--fs-error);
+ font-weight: var(--fw-error);
text-align: left;
- margin-left: 24px;
+ margin-left: var(--adjustment-left-field-and-error);
- margin-top: 8px;
+ margin-top: 0.5rem;
}
.error-hidden {
opacity: 0;
}
+.error-visible {
+ opacity: 1;
+}
+
/* Screen Reader only */
.visually-hidden {
border: 0;
@@ -196,7 +207,7 @@ main {
white-space: nowrap;
}
-@media screen and (min-width: 1000px) {
+@media screen and (min-width: 62.5rem) {
:root {
--fs-header: 4rem;
--lh-header: 100%;
@@ -209,38 +220,33 @@ main {
--lh-field: 175%;
--lh-error: 215%;
+
+ --adjustment-left-field-and-error: 2rem;
}
body {
- background-color: var(--gradient-white);
- background-image: url(../images/bg-pattern-desktop.svg);
- background-repeat: no-repeat;
- background-blend-mode: color;
- background-size: auto;
- /* background-position: left; */
+ background: var(--gradient-white);
- margin: 0 auto;
+ margin: auto;
}
main {
- max-width: 1440px;
- height: 100%;
+ background-image: url(../images/bg-pattern-desktop.svg);
+ background-repeat: no-repeat;
+ max-width: 90rem;
display: grid;
- /* grid-template-areas: "logo hero-image body"; */
+
grid-template-columns: repeat(2, 1fr);
grid-template-rows: 20% 80%;
grid-template-areas:
"logo hero-image"
"body hero-image";
-
- /* grid-auto-rows: auto;
- gap: 55px; */
}
.header {
margin: unset;
align-self: center;
- margin-left: 166px;
+ margin-left: 10.375rem;
}
.logo {
@@ -254,8 +260,8 @@ main {
.body {
margin: unset;
align-items: baseline;
- margin-top: 72px;
- margin-left: 165px;
+ margin-top: 4.5rem;
+ margin-left: 10.3125rem;
text-align: start;
}
@@ -264,7 +270,7 @@ main {
}
.body__text {
- margin-top: 18px;
+ margin-top: 1.125rem;
width: 45ch;
}
@@ -274,11 +280,15 @@ main {
.email-form__input {
width: 45ch;
- padding: 15px 0 15px 24px;
+ padding: 0.9375rem 0 0.9375rem var(--adjustment-left-field-and-error);
}
.email-form__button-submit {
- padding: 18px 45.1px 18px 46px;
- right: 110px;
+ padding: 1.125rem 2.8188rem 1.125rem 2.875rem;
+ right: 6.875rem;
+ }
+
+ .email-form__error-message {
+ margin-left: var(--adjustment-left-field-and-error);
}
}
diff --git a/projects/FrontendMentor/newbie/base-apparel-coming-soon/index.html b/projects/FrontendMentor/newbie/base-apparel-coming-soon/index.html
index 10d48b6..9f2cc90 100644
--- a/projects/FrontendMentor/newbie/base-apparel-coming-soon/index.html
+++ b/projects/FrontendMentor/newbie/base-apparel-coming-soon/index.html
@@ -51,11 +51,11 @@
Hello fellow shoppers! We're currently building our new fashion store.
Add your email below to stay up-to-date with announcements and our launch deals.
-
diff --git a/projects/FrontendMentor/newbie/base-apparel-coming-soon/js/script.js b/projects/FrontendMentor/newbie/base-apparel-coming-soon/js/script.js
index bcf3b34..3414a5f 100644
--- a/projects/FrontendMentor/newbie/base-apparel-coming-soon/js/script.js
+++ b/projects/FrontendMentor/newbie/base-apparel-coming-soon/js/script.js
@@ -1,4 +1,36 @@
"uses strict";
+const emailForm = document.getElementById("email-form");
const emailInput = document.getElementById("email");
const errorMessage = document.getElementById("errorMessage");
+
+emailForm.addEventListener("submit", checkEmailValidity, false);
+emailInput.addEventListener("focus", inputFocused, false);
+
+/**
+ * @param {[Event]} event
+ */
+function checkEmailValidity(event) {
+ if (emailForm.checkValidity() === false) {
+ event.preventDefault();
+ emailInput.classList.add("error-border");
+ errorMessage.classList.add("error-visible");
+ } else {
+ emailInput.classList.remove("error-border");
+ errorMessage.classList.remove("error-visible");
+ }
+}
+
+// When an input has focus I remove the error state,
+function inputFocused(event) {
+ /**
+ * Just for VS Code intellisense
+ *
+ * @type {HTMLInputElement}
+ */
+ const input = event.currentTarget;
+ if (input.classList.contains("error-border")) {
+ input.classList.remove("error-border");
+ errorMessage.classList.remove("error-visible");
+ }
+}