@Miculino
Posted
Congrats on completing the challenge, @Sneflenie!
It looks really nice and very similar to the original design. Also, all the features work as expected and there are no real issues with your responsive design.
Just a few suggestions:
-
The card is missing a box-shadow as the one in the original design
-
The edges of the .plan-info div might be a bit too round
-
Try reducing just a bit the space around the content inside the card
-
On larger screen sizes, the background image doesn't stretch fully to cover the entire viewport width
-
Don't use fixed values for your widths and heights. As a rule of thumb, try to always build your layout around the content you have available.
-
Try to use less of the px unit and more of the relative units such as rem, em, vh / vw, %
Hope this helps. Keep up the great work!
Marked as helpful
@Sneflenie
Posted
@Remus432 Awesome, Thank you for the tips! I will try my best to implement them. :)
