Latest comments
- @ChristianM-starSubmitted 8 months ago
- @Veena-K-VenugopalSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
I am proud of creating a solution similar to the preview even without exact design measurements.
What challenges did you encounter, and how did you overcome them?Setting the style to highlight the list of links on hover was a bit challenging as the text color was still white even when I added the required color to the li:hover selector. I fixed this by adding color:inherit to li:hover a selector.
- @olegtremba1234Submitted 8 months ago
- P@cacesasaSubmitted 8 months agoWhat are you most proud of, and what would you do differently next time?
I'm proud that I was able to completed the challenge.
What challenges did you encounter, and how did you overcome them?None for this one.
What specific areas of your project would you like help with?I will like to keep learning CSS.
- @Duced74Submitted over 3 years ago@luis08201Posted over 3 years ago
Hi @Duced74, You're doing an amazing job, Keep trying.
I saw in your code that you did this page using Bootstrap. Let me give you a feedback:
Remember how grid system works:
<div class="container"> <div class="row"> <div class="col-sm"> One of three columns </div> <div class="col-sm"> One of three columns </div> <div class="col-sm"> One of three columns </div> </div> </div>
I figured out you coded grid systems using row.
<div class="row"> <div class="row"> <div class="col-6"> One of three columns </div> <div class="col-6"> One of three columns </div> <div class="col-sm"> One of three columns </div> </div> </div>
replace first
<div class="row"></div>
to<div class="container"></div>
I think it is better to specify the column size. Both col were coded
<div class="col-6">
replace to<div class="col-xs-12 col-sm-6">
Just one more thing Naming classes according to content
<div class="card-features"> <div class="card-companies"> 10k+ <p>COMPANIES</p> </div> <div class="card-template"> 314 <p>TEMPLATES</p> </div> <div class="card-queries"> 12M+ <p>QUERIES</p> </div> </div>
I hope this helps you.
Regards.
Marked as helpful0 - @azzykesumaSubmitted over 3 years ago@luis08201Posted over 3 years ago
Thank you very much, I based on your project to make the switch color to the calculator. Regards
Marked as helpful0