@0xabdul
Posted
Hello Developer well congratulations on successfully completed the results summary component
- A Some suggestions for improve your code
- In Html π :
- LANDMARKS π
- the html page should be contained
one main landmark
- to clear the Accessibility reports use sematic elements or non semantic elements..
- Semantic elements βοΈ
- <aside> <nav> <section> <main> <header> <footer> <article> Ect..
- using semantic elements
- Ex :
<html lang="en">
<head>
<title>my page</title>
</head>
<body>
<header>
//logo img or heading ect...
//Using heading only
</header>
<main>
//Use only main of the contents
</main>
<footer>
//use copy write discription or challenge d by front end mentor code by mark..
</footer>
</body>
</html>
- HEADING β οΈ
- The page must contain at least one h1 element identifying and describing the main content of the page. An h1 heading provides an important navigation point for users of assistive technologies, allowing them to easily find the main content of the page.
- So fix them
- Ex :
<h1> </h1>
<h2> </h2>
<h3> </h3>
- Note : we use the headings line by line in order way
- In Css π¨ :
- And your results summary component is not center aligning so fix the problem
- Ex :
body {
display:grid;
place-items:center;
margin:0;
height:100vh;
}
- Now center aligning π―
- I Hope you find the solution and it's useful comment for you your results summary component project doing very well Happy Coding Developer
Marked as helpful
@IpieA
Posted
@0xAbdul Thanks for your feedback Abdul. It's much appreciatedπ. I'd work with your suggestions to improve my code