NW6 | Fathi_Kahin | HTML-CSS | Multipage-clone | Week 4 #178
NW6 | Fathi_Kahin | HTML-CSS | Multipage-clone | Week 4 #178
Conversation
✅ Deploy Preview for cyf-module-html-css ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
eliza-an
left a comment
There was a problem hiding this comment.
Well done on the use of animations and making the image look similar. Your use of variables in css to save the colors is very impressive and I am very happy to see so much progress!
Unfortunately, you are missing a very big part of the assignment here. You design had to be responsive. This means that when I look at it on a mobile device, it will still look like the template. Try to fix this using media queries as this is what happens when I view the page in responsive mode :

To check if your page is responsive, inspect the page and click this button


I have added a few resources already, but here is the media queries again: https://www.w3schools.com/css/css_rwd_mediaqueries.asp
| <header> | ||
| <section class="top-navigation"> | ||
| <section class="input"> | ||
| <input |
| <span></span> | ||
| <span></span> | ||
| </div> | ||
| <ul class="list tabs"> |
| @@ -1,3 +1,771 @@ | |||
| /* Add your CSS here */ | |||
There was a problem hiding this comment.
Notice how one of the main requirements is that you have "a mobile first responsive design".
This means that your page must look like the template on a phone as well. This is what yours looks like :

Do you see how there are quite a few differences? Try to fix this when you get the chance ! You can do this using media queries. Here is a link : https://www.w3schools.com/cssref/css3_pr_mediaquery.php
| </svg> | ||
| <span>2035</span> | ||
| </div> | ||
| <h1 class="heading">PRICKLES & CO</h1> |
There was a problem hiding this comment.
Your text is not centered here! How can you fix this?
| </div> | ||
| <h1 class="heading">PRICKLES & CO</h1> | ||
| <p class="hero-card-p">BRING NATURE INDOORS</p> | ||
| <button class="hero-btn">Shop Now</button> |
There was a problem hiding this comment.
When i hover over the shop now button on the template, this is what i see

But on yours we have this:
Here is a usefuol link : https://www.w3schools.com/cssref/sel_hover.php
| </section> | ||
| <section class="columns"> | ||
| <section class="about-us" id="about"> | ||
| <div class="about-us-card"> |
| Prickles & Co. can make your home more lush and vibrant. | ||
| </p> | ||
| </div> | ||
| <div class="/about-us-img-1"></div> |
| <div class="buy-online-img"></div> | ||
| <div class="buy-online-card"> | ||
| <p class="buy-online-p">BUY ONLINE NOW & GET 10% OFF !</p> | ||
| <button class="hero-btn buy-online-btn">Shop Now</button> |
| </section> | ||
| <section class="contact-us" id="contact"> | ||
| <form class="contact-form"> | ||
| <h2 class="about-us-heading">FOR SPECIAL REQUESTS & ORDERS</h2> |
| >First Name | ||
| <input type="text" name="first-name" id="f-name" /> | ||
| </label> | ||
| <label for="last-name" |










Learners, PR Template
Self checklist
Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.