<html lang="en">
Everything looks good to me here! Although not gonna lie and its probably minor but not using lorem ipsum, and having all those "xxxxxxxxx" does make it slightly distracting
31 Matching Annotations
- Nov 2022
-
github.com github.com
-
-
<h3>Latest News</h3> <p><b>October 1:</b><br />xxxxxx</p> <p><b>September 15th:</b><br />xxxxxx</p> <p><b>September 10th:</b><br />xxxxxx</p>
I think there would be a better way to do this than using many line breaks. Maybe display: block? Then give the div a max width. Just thinking but i'm not 1million percent sure
-
-
github.com github.com
-
/* Background-color doesn't work and I don't know why!!!! *crying*/
I'm trying to figure this out for you! but i can't and now im *crying * tooooooo
-
-
github.com github.com
-
parent { display: flex; } .child {
Using classes that better describe the content time will make it easier to understand the CSS. Maybe instead of parent and child, it could be something like "project example 1".
-
-
github.com github.com
-
<h3>Follow Us</h3> <ul class="list-2"> <li>Facebook</li> </ul> <ul> <li>Instagram</li> </ul> <ul> <li>Youtube</li> </ul>
I don't think you need to start a new un orderdered list for each item. I think it would be okay to have it as one ul, then have facebook, instragram and youtube as list items under that one list. You gave this the class list-2 but the way you did it, it only targeting the first item with that class.
-
<h1> Home Services Portfolio Contact</h1>
I think this would belong in a nav and ul element with each link being a list item , rather then an h1!
-
- Oct 2022
-
github.com github.com
-
article h1 {
I don't think you'd have to specify to target all h1's within an article. You should be able to apply this style to the single element h1, as there should be only one h1 on a page!
-
(images/noelle-rebekah-9XfvYJwR-SA-unsplash.jpg)
I believe you're missing double quotes here around the location of the image :)
-
-
github.com github.com
-
<article> <h1> <u> Assignment E - Website evaluation </u> </h1> <p> I have chosen the Jim’s clothes closet website to prepare my evaluation on. <br>I believe there is plenty that could be improved on this website. </p> </article>
I don't know if an article element would make the most sense to have the h1 of the page. Maybe in a header, or even main?
-
<u> Assignment E - Website evaluation </u>
I'm not absolutely sure but does 'u' element create an underline effect? If so, perhaps using the text-decoration property and setting to underline might be a better way of applying a style. Underlines are usually meant to indicate a clickable link.
-
-
github.com github.com
-
<h2 class="card4_h2"><span class="h2_part1">Live</span> <span class=h2_part2>Breaking</span> <span class="h2_part3">UK</span></h2>
Bravo for using span here!
-
<p> 'Plan C' rumours grow;<br>Downing Street Christmas party cancelled; warning Scotland's R number<br> could rise above 2 | COVID<br> latest </p>
I think this would make more sense as a header as its the most important information on the card! Also, changing margins and would be more appropriate to create space over using <br> elements!
-
<span class="firstbullet">•</span> • •
Empty span elements would work great to make bullets here!
-
-
github.com github.com
-
0px;
Pixels isnt required when the value is 0. You might get errors in your validator for this! Although I'm not positive.
-
box-sizing: border-box; width: 560px; background-color: hsl(234, 12%, 34%); margin: 50px; padding: 30px; border-radius: 30px; background: rgb(194,241,0); background: linear-gradient(90deg, rgba(194,241,0,1) 0%, rgb(7, 0, 0) 49%, rgba(0,212,255,1) 100%); /*copied from https://cssgradient.io/ */ background-image: linear-gradient(90deg, rgba(193, 241, 0, 0.425) 0%, rgba(7, 0, 0, 0.432) 49%, rgba(0, 213, 255, 0.342) 100%), url(images/elephants.jpeg); /* img copied from https://unsplash.com/ */ background-size: 560px 300px;
It seems you have multiple background colours here. You have selected background colours. One as 'background-color' and one as 'background'. As well, it looks like you have two sets of gradients. Was this intentional?
-
-
github.com github.com
-
<!DOCTYPE html>
Your HTML looks good to me.
-
-
github.com github.com
-
<p>Below is a list of services we proudly offer. At Hairpins, we care about the environment and recognize the impact we all have on it. We are continually making strides to reduce where we can and have only aligned ourselves with companies and products we believe in. We are proud to be a CERTIFIED GREEN CIRCLE SALON and through that partnership are able to divert 95% of our salon waste from landfills. Get in touch if you have any questions or want to learn more about the programs and charities we are focusing our efforts on.</p>
I might add a few like breaks in here making it easier for the viewer to read!
-
-
github.com github.com
-
nav ul {
Your CSS looks good to me!
-
-
github.com github.com
-
Nothing stands out as incorrect to me!
-
-
github.com github.com
-
<break>All cuts include shampoo, scalp massage, blowdry, and style</break>
Was this meant to be a break element as in <br>? If not, it may just be another element i'm unaware of ! :)
-
-
github.com github.com
-
class="oswald">*
I would suggest not naming classes after font families in the chance you might want to change the font in the future. You would have to change all the classes as well.
-
-
github.com github.com
-
id="spacedletters"
I would use an ID that isn't named after the stylistic choice of the ID. Perhaps something more vague like "Address" would make sense, incase you decide not to space out the letters later.
-
<figure class="oswald"><ul>
I thought the figure element was meant for images. I wonder if this might have a better option to box it out. Im not exactly sure.
-
- Sep 2022
-
github.com github.com
-
<!DOCTYPE html>
I'm having a hard time seeing anything wrong with your code! I've read it over a few times and at the moment nothing stands out that I can see needs to be fixed!
-
-
github.com github.com
-
<h2>How much web experience do i have?</h2> <p>I have no experince in web development but i always like to push myself and decided to learn tjhis subject.</p> <h2>Do you know any coding languages?</h2> <p>No, i do not have any knowlegde regarding coding languages.But i came here to learn more i am trying to make this language work for me.</p> <h2>Your expection from this course?</h2> <p>My expections from this course is that i should give me some new experience and learnig in the
I'm not sure how to explain this but I think It would be easier to read and understand hierarchies here if you spaced and indented the lines and did the formatting more accordant to the content. I hope this makes sense but I'm not sure yet how to describe that.
-
<h2>Who am I</h2> <p>Hey, My name is <em><b>Nempal Singh</b></em> and i am currently learning DGL 102. I am from India and i moved to Canada in summer of 2021.</p> My favourite things that i often find time to do: <ul> <li>Playing footbal</li> <li>Watching movies</li> <li>Reading books to develop my mind</li> </ul> <p>Here is my contact info, if you want to reach ouut to me.</p> <ul> <li><a href="mailto:nempalsingh1307@gmail.com"> nempalsingh1307@gmail.com</a></li> </ul> <h2>How much web experience do i have?</h2> <p>I have no experince in web development but i always like to push myself and decided to learn tjhis subject.</p> <h2>Do you know any coding languages?</h2> <p>No, i do not have any knowlegde regarding coding languages.But i came here to learn more i am trying to make this language work for me.</p> <h2>Your expection from this course?</h2> <p>My expections from this course is that i should give me some new experience and learnig in the world of websites and web development.</p>
I feel like this might belong in within a semantic structure, such as main or section! To add semantic value and style easier.
-
-
github.com github.com
-
<!DOCTYPE html>
Not incorrect, but wordwrap may make it easier to work on! Made it easier for me !
-
<p style=color:green>
I believe styling should be in CSS, unless this has other meaning that I'm not aware of! :)
-
-
github.com github.com
-
<!DOCTYPE html>
I don't see anything wrong with your code!
-
-
github.com github.com
-
<!DOCTYPE html>
I don't any issues with your code, but might recommend the use of capitals in your Title, headers and paragraphs :)
-
-
github.com github.com
-
<!DOCTYPE html>
I've looked over your code, and I don't see any issues!
-