@dewslyse
Posted
Hello @esthercate! Congrats on your submission. Your solution is very well implemented. Good job.
- You could center the card to the page by adding the following to the
body
in css.
body {
display: flex;
flex-direction: column;
min-height: 100vh;
justify-content: center;
align-items: center;
}
- You could also add semantic sectioning elements to your html. You could change
<div class="hero-section">
to<header class="hero-section">
,<div class="main">
to<main class="main">
and maybe<div class="attribution">
to<footer class="attribution">
. This will help address the accessibility issues raised in the report.
Happy coding
Marked as helpful
@esthercate
Posted
@dewslyse Thank you. I really appreciate your feedback.