Latest solutions
Latest comments
- @IdodeHaan
- @migmcc@mrjayy5
Hi Migmcc,
Your HTML structure is perfect. No comment about that 👏🏼
You probably need the most work on your CSS.
- CSS Introduction. This site has a lot of information that will help with your CSS skills.
- This complete guide to Grid might help you as well.
I hope you can find some solution to make this project even better!
Keep on coding 🔥!
- @deanhopes@mrjayy5
Hi Dean, great job! It's even a match without PixelPerfect 👏🏼!
Your HTML is semantic almost perfect. The only suggestion I have is that you use
h3
beforeh2
. I made this mistake myself sometimes!Also there is still a little bit of redundant code. You might find this usefull:
<header class="singleprice__header"> <h2>Monthly Subscription</h2> </header> <header class="singleprice__header"> <h1>Join our community</h1> <h2 class="yellow">30-day, hassle-free money back guarantee</h2> </header> .singleprice__header { h1 { ... } h2 { ... &.yellow { ... } } }
Nevertheless, great work! Keep on coding 🔥!
- @darrenevans81@mrjayy5
Hi Darren,
How was working on your first project?
I think you did a great job! Working with Bootstrap is (still) a nice to have for a lot of employers. But for small projects I think it's a downside, because it's has to load the entire bootstrap framework and you will only use a small part of it.
You can speed up this page by using only Gridbox and Flexbox instead of bootstrap.
The only thing I can suggest on your code is your indent. A good use of indent ensures a readable structure. Minor indent mistakes in your CSS as well.
Keep on coding 🔥!
- @Yemisrach15@mrjayy5
Hi Yemisrach, I love your work 👏🏼!
You have a great eye for detail I can see 😉! I like to see how you handled the responsiveness, great work!
-
I think you should try to make a
.star
element in your CSS and replace theimg
's with<div class='star-icon'></div>
. This way the page only need to download the SVG once. But you still have to insert the element 5 times. -
I think the HTML is pretty clean. Good indent, so it's nice to read. You should consider using Sass. It can reduce the lines of CSS significantly and it's easier to maintain. So especially for bigger projects it's a must have!
Keep on coding 🔥!
-
- @Senpai013045@mrjayy5
Hi Senpai! Great work, the API is working 👏🏼!
I can only suggest to spend a little bit more time on your
margin
s,padding
s andfont-size
. It will help to make this project looks more like the design.Keep on coding 🔥!