Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

3 Columns Cars

Fab29012 60

@Fab29012

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


This was really fun to do ! Please don't hesitate to comment on how I could improve !

Community feedback

Daniel 🛸 44,230

@danielmrz-dev

Posted

Hello @Fab29012!

your project looks great! I see that you added your special touch to it!

I have a suggestion for you to improve it:

Since all the 3 card share these properties:

    border: 1px solid transparent;
    text-align: center;
    height: 400px;
    width: 200px;
    border-radius: 5px;

You could've created new a class with them and add this new class to each card. This way you wouldn't have to repeat that 3 times. The elements can have more than one class.

So, you can do that:

.new-class {
    border: 1px solid transparent;
    text-align: center;
    height: 400px;
    width: 200px;
    border-radius: 5px;
}

And then add the class to each card. That's it!

I hope it helps!

Marked as helpful

1

Fab29012 60

@Fab29012

Posted

Hey @danielmrz-dev !

Thank you so much for your feedback ! I'll definitely try to remember to implement it into my future works ! That would help clean up my work and allow me to keep track of my styles ! Thank you !

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord