Samson Shamβ’ 140
@samsonsham
Posted
Hello Facundo π Congratulations for completing this challenge π I see you make good use of grid box and calculate the row and column nicelyπ
Some of my suggestions below:
- I see you have imported the font family Poppins but I couldn't see you are using it.
font-family: 'Poppins', sans-serif;
So you might not be intentionally not using the given font, right? - I see you put the first part of heading into
<span>
. It should be part of the heading so is semantically better to put it intoh1
as well and add necessary<strong>
and line break. And<span>
is an inline element so be careful your using might cause some unwanted inline effect. - Incorrect icon in Calculator card.
Hope this helps! Have a nice day!
Marked as helpful
0
Facundo Rubioβ’ 250
@Facu3071
Posted
@samsonsham Thanks you so much for you help. Good vibes Samson!
1