Skip to content
OverviewSolutions
31
Comments
7
Harold
@harika09

All comments

  • Harold•628
    @harika09Posted almost 4 years ago

    Good job on this one.

    some recommendation to improve it.

    1. try not fix the illustration image when display on bigger sizes display it will zoom it. 2.add spacing on your navigation menu. 3.add spacing on the latest articles card.
    2. make it responsive.

    overall this was great! Happy coding!

    0
  • Gerri Ezeocha•460
    @GerriEzeochaSubmitted almost 4 years ago

    This project was my first exposure to FrontEnd Mentor! I worked through the YouTube Series "Build a website from scratch" by Jessica Chan. She is a great teacher. I had some issues loading my images for my hero section, I had to move them to the same root folder holding my sass/style sheets.

    All in all, I learned A LOT working on this project, and hopefully can apply those to future projects.

    Easybank Landing Page

    2
    Harold•628
    @harika09Posted almost 4 years ago

    Great job on this one @Gerri Ezeocha.

    just to remove the min-height: 350vh; to remove the white spaces after the attribution.

    Happy Coding!

    0
  • Harold•628
    @harika09Posted almost 4 years ago

    Great Job on this one @Stelin

    Got some few recommendation.

    1. check the validation when I clicked the button it will just stack the error.
    2. add cursor: pointer; when hover on the button.
    3. lower the padding on the smaller device on class="info".

    Good job on this. Happy coding!

    1
  • Harold•628
    @harika09Posted almost 4 years ago

    Nice work with this. I will just add some recommendation.

    1. try to center the illustration-img to center with it its max-with: 900px
    2. try to center to footer just like the design.

    Happy Coding!

    0
  • Harold•628
    @harika09Posted about 4 years ago

    Nice work with this.

    try to add the details, box and like on a container Example <main class="container"> <div class="details">...</div>

    <div class="box">...</div> <div class="like">...</div> </main>

    also add no-repeat on the background image

    then on the css add 100vh on the main

    0
  • Harold•628
    @harika09Submitted about 4 years ago

    accepting feedback specially the image background suggestions

    HTML 5 & CSS

    3
    Harold•628
    @harika09Posted about 4 years ago

    This is noted. Thanks!

    0
  • Harold•628
    @harika09Posted about 4 years ago

    not expert here but i suggest to remove the margin from article__title and lower the margin from article__snipet to 10 then lower the padding from article__content to 10 top to fix the display.

    1