font-size: 62.5%;
Looks good.. all the styles are achieved according to the reference image
font-size: 62.5%;
Looks good.. all the styles are achieved according to the reference image
<span class="btn">Get in touch</span>
use type ="button" as given in readme file
Copyright ©2022. All rights reserved North Island College Digital Designn + Development
In my opinion footer font size can be reduced to small when compared to the reference page given
gal gal3
change the class name to meaningful one
<p>
The analysis was good and design was attractive. You have put extra effort to make the mockups. And one suggestion is to add images below the issues mentioned because it is easy to understand
prices
Naming conversions are good and styles are well- formatted
<link rel="preconnect" href="https://fonts.googleapis.com"> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> <link href="https://fonts.googleapis.com/css2?family=Alkalami&family=Oswald:wght@500&family=Poppins:wght@200&display=swap" rel="stylesheet">
Combine all google fonts into because to avoid repetition of code
.dot
Looks good and added all styles which is mentioned in instruction
<link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> <link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> <link href="https://fonts.googleapis.com/css2?family=Milonga&display=swap" rel="stylesheet" /> <link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> <link rel="preconnect" href="https://fonts.googleapis.com"> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> <link href="https://fonts.googleapis.com/css2?family=Open+Sans:wght@300&display=swap" rel="stylesheet"> <link href="https://fonts.googleapis.com/css2?family=Brygada+1918:ital@1&family=Milonga&display=swap" rel="stylesheet" /> <link href="https://fonts.googleapis.com/css2?family=Brygada+1918:ital@1&family=Jost:wght@300;400;500&family=Milonga&family=Ubuntu:wght@500&display=swap" rel="stylesheet" /> <link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin /> <link href="https://fonts.googleapis.com/css2?family=Brygada+1918:ital@1&family=Milonga&family=Ubuntu:wght@500&display=swap" rel="stylesheet" /> <link href="https://fonts.googleapis.com/css2?family=Radio+Canada:wght@700&display=swap" rel="stylesheet" /> <link href="https://fonts.googleapis.com/css2?family=Poppins:ital,wght@0,200;0,400;0,600;1,200;1,400;1,600&display=swap" rel="stylesheet" />
Repeated font family import . you can copy all to one line
silk
Naming conversions are good enough.
card-three
Card-four css are missing.
.lionimg h2
font-family is not assigned to the css
</article>
Card four is missing.
styles.css"
Different reference file name for css. Actually you have style.css, but you write styles.css in all page
</head>
There is no embedded style ,since in Step 3 it is mentioned to use embedded styles
header
Doesn't add any background picture ,it is mentioned in instructions
p .bookappointment
Haven't changed the footer color
background: blue; }
Used color name but not any hexadecimal value of color in style
blockquote
Haven't used any Id or Class selectors
<p>
No inline styles in any of the page
/>
There is not embedded styles
<P>I
please make uppercase
to lowercase for all paragraphs
<Header><h1><strong>ASSIGNMENT B</strong></h1></Header>
great work.Looks fine
Digital Design + Development (Web development)
use keyword for highlighting your course
<h2><b>Who am I?</b></h2>
do not use strong inside h2 element
<h1><strong>Assignment - B</strong></h1>
do not use strong inside h1 element
Hello Digital Design And Development
make it as comment or write inside an element
<h1>Hello World</h1> <p>This is *Aran Arora* speaking.</p>
Make the elements in 2 lines as it is easy to read (elements h1 and p)