3-column-preview-card-component-main

Community feedback

• 

  P

  Teodor Jenkler•3,820

  @TedJenklerPosted 3 months ago

  Hi @Gayatri0109,

  Nice project! Here are a few suggestions for improvement:

  Avoid Using IDs for Styling: I noticed you’re using IDs frequently in your project. It’s generally better to use classes for styling instead of IDs. Classes are more reusable and maintainable, which makes your CSS easier to manage. Additionally, using IDs can lead to CSS specificity issues. Also, consider learning Flexbox for more flexible and easier layouts.

  Add ARIA Labels: To enhance accessibility, consider adding ARIA labels to describe the content of your <div> elements. This practice is particularly useful in larger projects, as it helps screen readers understand the purpose of each element.

  Hope these suggestions are helpful!

  Best, Teodor

  Marked as helpful

  0
• 

  Emre Söylemez•220

  @soymzePosted 3 months ago

  Hey, your solution looks pretty same with the design.There are only 2 things that I saw as a problem.First one is positioning.Your card is not in the middle,It is in the way more right side of the viewport and I checked your css file.I think if you use percent values instead of pixels, the card is going to be more responsive.The other thing is when I previewed your solution, one of the buttons were out of the div.So maybe you can use flexbox for that.Cause I coded this design yesterday and had the same problem.The length of p elements are different and because of that alligning buttons is hard.Happy coding :)

  Marked as helpful

  0