@tomazcx
Tarek Mo
@tarek-moAll comments
- @tarek-mo
It seems like you forgot the background in the "Boost your links today" section as well as the shorten link section. But everything else looks good man!
- @purnimakumarr@tarek-mo
To remove that space under your image use
display: blockon your image tag, the reason behind this issue is that images are treated as text so the bottom is to accommodate for those pesky: "p,q,y,g,j" letters. - @claudiabdm@tarek-mo
Damn, you really got it. great animations ^^
- @Evavic44@tarek-mo
Hey Victor, I've read your code and this is what i think could be improved: the section with the class 'container' isn't perfectly centered vertically, and that's because you added a margin top of 10.5rem. Instead what you can do is set this section to position absolute and type top: 50%; left: 0; transform: translateY(-50%) which will center it vertically in all devices
Marked as helpful - @DiegoVelazquez60@tarek-mo
Hello DIEGOVELAZQUEZ60,
Everything is looking good, one thing I've noticed (and you probably did aswell) is the annoying sidescrolling that was caused in your page. You can add
overflowX: hiddenin your Css body selector and that'll fix this issue. Good Luck! - @AlMon@tarek-mo
I thought react is only used for big web apps, have no experience with it since I'm still learning javascript. But hey, your website is looking professional. Good job.
- @mdagudo@tarek-mo
hello, you can use Bootstrap's grid system to make this easily responsive. If you never used Bootstrap and don't want to yet. you can use flex wrap and set it to wrap maybe? or use display grid with the minmax function. I've been using Bootstrap a lot lately so i kinda forgot how i used to layout with only CSS LOL. good luck!
- @matthew-io@tarek-mo
very aesthetic and clean look , good job
