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

Pure css

Raul 290

@Raul-code1

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Kyle 80

@KylesTech95

Posted

Hi Raul,

Great job on this project! Although your HTML & CSS look sharp, there is always room for improvement.

  • I understand that the goal was to solely style your project, however, the element used for the ["price"] class seems to be preventing the $169.99 from looking like the preferred font-size in the desktop design.

  • Alternatively, I wrapped both contents separately, one for [h2.discounted-price] & one for [p.original-price]. Both elements are styled by setting their display to inline, and then styling both selections individually. Please view the changes made below:

<h2 class="discounted-price">$149.99</h2>
<p class="original-price">$169.99</p>
/*added inline-block to place the h2 & p tags side-by-side*/
.discounted-price, .original-price {
/*color: var(--Dark-cyan);
font-size: var(--font-fraunces);
font-size: 30px;
font-weight: 700;
padding-bottom: 22px;*/
display: inline-block;
}
h2.discounted-price {
font-family: var(--font-fraunces);
font-size: 30px;
color: var(--Dark-cyan);
}
p.original-price {
color: var(--Dark-grayish-blue);
text-decoration: line-through;
padding-left: 2rem;
}
  • When it comes to flexbox, it is crucial to understand what is being flexed and how certain elements respond to flexbox. Rather than setting the flex direction to column, set the flex direction to row by deleting the column attribute. Reason being is that flex-row will span both [div] elements side-by-side. In other words, the "hero-image" will be right next to "card-info". Please view the changes made below:
.container {
display: flex;
max-width: 100%;
max-height: 55%;
margin: 0 auto;
padding: 0;
border-radius: 10px;
overflow: hidden;
}

I hope this helps!

Marked as helpful

0

Raul 290

@Raul-code1

Posted

@KylesTech95 Thanks for the tips :D . i put flex direction column because i started with mobile view first

1
Kyle 80

@KylesTech95

Posted

@Raul-code1 I totally understand. By the way, I like the use of your media queries. very creative!

0
Raul 290

@Raul-code1

Posted

@KylesTech95 Thanks :)

0

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