@Karag2006
Posted
Hi, First off: i like your Solution, espacially the additional hover on the Summary Categories.
Now for some small suggestions: You don't use Semantic HTML. Which is an easy thing to make the Solution a little better still:
- Include a <main> tag fro the majority of your html. For Example your card-container :
<main class="card-container"> ..... </main>
- Also The provided attribution could be a footer instead of a div :
<footer class="attribution"> ..... </footer>
- You are missing an h1 Headline. Idealy that would be the main Thing telling people what the site is about. In this case that would be
<h1 class="txt-resultado">Your Result</h1>
And the last thing: you do have a very slight visual effekt on the <button> for focus. This could be more obvious so people prefering the keyboard have a better visual clue as to what Element has currently the Focus. What i usually do for that is :
.btn-continue:hover, .btn-continue:focus { background: linear-gradient(to bottom, var(--color-light-royal-blue), var(--color-light-slate-blue)); }
This just gives the button the same effect for hover and focus states.
I hope the Feedback is Helpful for you.
Marked as helpful
@daaaan12
Posted
@Karag2006
Hello, thank you very much for the feedback. I will start using more Semantic HTML. I am practicing every day to improve my skills and tips are always welcome. Thank you!