Tesla_Ambassador• 2,980
@tesla-ambassador
Posted
Hey, congrats on submitting your first solution! It's pretty awesome too! I just have a few issues to point out;
- Incase you want to get rid of a repetitive background like the way it is with the div containing your hero image, just set the "background-repeat" property in your css to "no-repeat".
- I noticed it wasn't responsive too just like you stated in your comments. But it's good coding practice to continuously resize your screen and apply media queries when items lose their alignment. It makes it super responsive.
- If you want to use some attributes more than once, consider using classes instead of ID's to avoid duplicate id's and this will eliminate your html issues.
Marked as helpful
2
Shreyansh shah• 45
@Shreyansh-07
Posted
@tesla-ambassador Okay so now I have fixed it many issue's and added flexbox. But as you can see the latest screenshot generated by frontentmentor have a problem with height or padding of the card but that problem doesn't show up on my tablet when I preview it in desktop mode. I tried to find what's doing this but no results any solutions ?
0
