RickLearnsToCode
@RickLearnsToCodeAll comments
- @Mithungowda18@RickLearnsToCode
Hi Mithun,
When comparing your version to the design, I notice you did not change the font color of your location. You might consider doing that to create the contrast between the name, location, and slogan at the top to make it more visually interesting.
Also, I initially used <button> elements in my attempt, but I did not like the default border and box-shadow added by the browser. Instead I changed <button> to <div> for each link then styled the divs accordingly. This gave them the flatter look shown in the design.
Nice job completing the challenge!
- @CodeWithRonnieWhat are you most proud of, and what would you do differently next time?
I'm proud of the fact that I did not have to look up the document or google on how to solve this challenge.
What challenges did you encounter, and how did you overcome them?I did not encounter any challenges.
What specific areas of your project would you like help with?Web Responsiveness. Still struggling with that.
@RickLearnsToCodeThis looks really good! Your font weights and colors are matching the design really well - I struggled with mine a bit. My only feedback is I can see you're using a bit of blur on your drop shadow, which the design doesn't seem to be using. Also, I think if you want to center the card vertically on the page you can get rid of the margin-top/margin-bottom settings on your .container class selector so the
margin: 0 auto;
setting above them will be used. - @youns-nibras@RickLearnsToCode
Nice implementation, looks just like the challenge examples!