Skip to content
OverviewSolutions
32
Comments
17

Philemon Bahati

@bahati7Congo620 points

Web Developer specializing in front end development. Experienced with all stages of the development cycle for dynamic web projects. Well-versed in numerous programming languages including HTML5, PHP, JavaScript, CSS, MySQL.

Latest solutions

View more solutions

Latest comments

  • Philemon Bahati•620
    @bahati7Posted about 2 years ago

    Any feedback is welcome:)

    0
  • Philemon Bahati•620
    @bahati7Posted over 2 years ago

    Hi @KEISE!

    I like your solution:)

    Just wrap your ".container" class in a "main" tag for accessibility

    Good job

    Marked as helpful
    0
  • Philemon Bahati•620
    @bahati7Posted over 2 years ago

    Hey @Joshua

    Your solution looks great! I like It :)

    Just wrap the "attribution" in the "footer tag" for accessibility.

    Your code is clean

    Thanks

    1
  • Philemon Bahati•620
    @bahati7Posted over 2 years ago

    Hey @anthony

    Good job on this! I'd like just to add few tips:

    first, to perfectly center the card, this code can help:

    .card{ position: absolute; top: 50%; left: 50%; transform: translate(-50%,-50%); }

    secondly, try to increase the width of the card on mobile and reduce the h1 size

    try until it's perfect

    good luck:)

    Marked as helpful
    0
  • Philemon Bahati•620
    @bahati7Posted almost 3 years ago

    Hello Yui,

    congratulations for this challenge:) just few tips to improve the code:

    • add "alt" attributes on img tags for accessibilty;
    • wrap your code in a "main" tag instead of div class="content">
    • use relative units like %, rem to specify the width of the ".content" class, after that you can add "max-width: 450px";
    • you are not obliged to specify the height of the ".content" class

    once again good job!!!

    0
  • Philemon Bahati•620
    @bahati7Posted almost 3 years ago

    Hello there:)

    You've done a good job on this; just some tips to improve the code:

    • wrap your code in a "main tag"
    • interactive controls (button, links) must not be nested like <button class="button"><a href="#" class="hash">Sign Up</a></button>. just use "a" tag.
    • try to use media queries for mobile design, there is a problem of responsiveness. @media (max-width:765px){...}

    Once again good job!!!

    Marked as helpful
    0
View more comments