Hey guys,
Looking for some feed back on this project:
Thanks for your time!
Cheers,
Hey guys,
Looking for some feed back on this project:
Thanks for your time!
Cheers,
Hey @kbventures! Nice tribute page. The requirements for this project aren’t very strict so my review will almost totally subjective.
What I like:
What I’d change:
It was a pleasure to review your project. Keep up the good work
Hello,
Thanks for taking the time to give some feedback. I implemented all changes. For the social icon there’s a color change on hover for the new SVG code.
I am not sure why the image for the “4 Hour Work Week” link to Amazon seems to take more than the image space… any clue how to fix that?
Cheers,
Your page looks good @kbventures. Some things to revisit;
@kbventures I’m getting back to web-dev myself, so my specifics may be a bit rusty, but the problem with the link seems to occur because of the default block display of its parent container. It works well for me if in your styles css page I customize the amazon-link class as following.
.amazon-link{
display: flex;
justify-content: center;
align-items: center;
}
I think because by default it has display: block, both the parent and children tend to stretch as much as possible. This is just an assumption though, more research is needed for a reliable answer.
Merry Christmas and keep coding .