- Nov 2022
-
github.com github.com
-
<html lang="en">
Images folder? it isn't necessary but looks cleaner.
-
- Oct 2022
-
github.com github.com
-
Content
-
<style> body { font-family: "Trirong", serif; font-weight:400;
I'm pretty sure you can add this to your style.css page as it's considered a css tag?. I think this is mixing html & css into one sheet which can become confusing.
-
br
The BR tag is already self closing, therefore you don't need to have a closing tag. A BR tag by it's self is perfectly fine.
-
Stylist
p tag!
-
Located
p tag, It'll still load properly without but it's needed.
-
-
github.com github.com
-
<h2><u>Colours</u> </h2> FULL FOIL: $188/$219 with cut <br> 3/4 FOIL: $172/$203 with cut<br> 1/2 FOIL: $158/$189 with cut<br> 1/4 FOIL: $144/$175 with cut<br>
(ul) (li)*(/li) (li) ****/li) (/ul)
-
hairpins
Capitalization! sorry for being nit-picky
-
and
Again, Maybe add a (p) tag as it's text.
-
-
github.com github.com
-
Located
I'm not to sure what happened here maybe it didn't go through when you committed? There's no (p) tag for the text here? there is for a couple of text's but some don't have anything? Key part!
-
- Sep 2022
-
github.com github.com
-
width
width=600 & height=??? make sure to add the height as well! This looks good otherwise!
-
-
github.com github.com
-
>
This looks good Rhiannon! There isn't actually anything to change! Just wanted to let you know on here
-
-
github.com github.com
-
p
I would put the
right beside the text for example
(p) Hi, My name is Melissa Jay. I reside in Victoria, BC. I am enrolled in a web design course at North Island College. (/ p)
Rather then above. I Just always keep it this way as it reduces confusion! Otherwise from that, It looks good!
-
-
github.com github.com
-
<img src="images/toy_car.JPG" width="600" alt="blue and green toy car">
I would add a height also in the future assignments, width = "600" height = "___"
-