@Adonay-Pro
Latest solutions
Latest comments
- @makjnr317
Hie there. Your site isn't showing. looks like you gave wrong link
- @abanicaisse@makjnr317
Hie there.
- You used the wrong link for the background, you should use
/images/pattern-background-mobile.svg - Also there is no need to add
#mainfor everything in your CSS, it just makes coding more boring since you are repeating the same thing. Same as.main__descriptionand.main__img, they do not need to be put on. - Instead of using
#mainin your CSS, you can still just saymain
- You used the wrong link for the background, you should use
- @caspercerete31@makjnr317
Hie there. Just wanted to say your website isn't loading the CSS
Marked as helpful - @morgan03@makjnr317
Hie there. Just a tip. Try putting everything under <body> into one container the use flex to center it.
Marked as helpful - @damaestro165@makjnr317
Hie there. Just a tip. Don't use margin to center <main> instead use flex. Also change the background for .containerText so that the whole page isn't blue. Besides that, most of the stuff is good.
Marked as helpful - @Ryoma1512@makjnr317
Most of it looks a good, only a few things to note.
- The text color on the Why Us section and part of the price section has to be changed as there is lack of contrast which makes it harder to read.
- Add a background color to the body.
- Add spread to the box-shadow so that it doesn't look almost as solid as a thin border.
- When in mobile view don't make the component take the whole body width.
Marked as helpful
