dewslyse• 3,025
@dewslyse
Posted
Hello Marialba! Congrats on completing another challenge. This is very good. Some suggestions for your consideration:
- You should set a
max-widthon your.container-card. I believe the the width for this component is635px. - You could also use
<li>for the why us section
<h3>Why Us</h3>
<ul>
<li>Tutorials by industry experts</li>
<li>Peer & expert code review</li>
<li>Coding exercises</li>
<li>Access to our GitHub repos</li>
<li>Community forum</li>
<li>Flashcard decks</li>
<li>New videos every week</li>
</ul>
- Remember to wrap the content within the
<body>in amaintag. - [
<button><a href="#">Sign Up</a></button>]<a>cannot be a descendant of<button>. Use one or the other.
Happy coding!
0
Marialba Córdova• 60
@Marialba16
Posted
Hello! Thanks again for your comment, I did not remember to use <ul>, now that you say it if it is more practical and feasible, and regarding the max-width I still do not understand it much and that is why I do not use it, but I'll start to put it into practice too. Thanks, again @dewslyse!!
0
