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

single-price-component using grid layout

rai-birajโ€ข 360

@birajrai100

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


welcome to all kind of correction and suggestions

Community feedback

Moโ€ข 840

@MohamedAridah

Posted

Hello @RaiPrayash, you did a Great job.๐Ÿ‘๐ŸŽ‰ and your solution looks similar to the design. But unfortunately it's Not Responsive. and i have some notes for you:

  • To center your .container element you can add these styles to the body:
.body{
	display: grid;
	place-items: center;
	min-height: 100vh
}
  • instead of set border-radius property separately for .box-1 , .box-2 and .box-3. You can just add it for the .container itself
.container{
	border-radius: 10px;
	overflow: hidden; /* to crop radius perfectly */
}
  • when using font-weight property don't use units. ex:

    • The wrong way

      	.element {
      		font-weight: 700px;
      	}
      
    • The right way

      	.element {
      		font-weight: 700;
      	}
      
  • row-gap property for .box-1 , .box-2 and .box-3 elements are quite big. Try to reduce it

  • add active state for button element on hover. it will be nice:

	.box-2 button {
		transition: 200ms linear; /* to make hover effect smoother */
	}
	.box-2 button:hover {
		background-color: hsl(71deg 73% 65% / 96%);
		cursor: pointer;
	}
  • For p element inside .box-3 p div. Try use ul element (unordered list) will be much better and this is More Html Semantic
	<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>ew videos every week</li>
	</ul>
  • Try to make your solution Responsive

I hope this wasn't too long for you, hoping also it was useful๐Ÿ˜.

Goodbye and have a nice day.

Keep coding๐Ÿš€

Marked as helpful

2

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