Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
0
Comments
7

Amir Jacobs

@Amir-Jacobs165 points

Software engineering student.

Latest solutions

No solutions submitted yet.

Latest comments

  • Guliye•290
    @guliye91
    Submitted over 4 years ago

    Responsive FAQ accordion using JS

    1
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    Hi Guliye

    I noticed a few things about your Javascript:

    • I'd use functions to make clear what is happening. You could make a function called resetAccordion() and a function called showAccordion().
    • I'd make a bit more use of comments.
    • The accordion can only be opened by clicking the icon, but it'd be more user-friendly to have it open upon clicking .question.

    Have a nice day. ✌

  • Rodrigo Gamboa•10
    @RodrigoGamboa
    Submitted over 4 years ago

    Profile Card Component using HTML and CSS

    2
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    Hey Rodrigo,

    I've taken a quick look and I noticed the following:

    • Instead of width: 375px I'd use width: 100%; max-width: 375px as this helps with responsive.
    • The blue background pattern images don't show up on mobile.
    • I'd add a margin to the sides of the page on mobile, so that the card doesn't touch the edges. Something like 5px to 15px should be perfect.
    • You could use the BEM method for your CSS. Not really necessary here as it's pretty clear, but I'd look into it nonetheless.

    Your code looks clean. Have a nice day. 😊

  • Bryan Martins de Araujo•50
    @bryanmaraujo544
    Submitted over 4 years ago

    Tracking Introduction

    2
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    Hey Bryan

    I've taken a look and I noticed the following things:

    • Your responsive design (376px and up) has some flaws.
    • You tend to use > in your CSS. I would make the elements specific by utilizing the BEM method.
    • You're using CSS variables -- that's great.

    Happy coding 😄

  • Karim•810
    @Karimsamir112
    Submitted over 4 years ago

    huddle-landing-page-with-curved-sections

    2
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    Hi Karim

    I took a quick look and I have some suggestions:

    • I'd stop using <br> to create space between elements. Instead, use margin or padding.
    • Your indentation could be improved -- perhaps look at how a code beautifier would indent your code.
    • In some cases I see inline-styling (such as <h1 style="font-size: 65px">2.7m+</h1>. I would put all styling in your CSS file instead.
    • Your class names and id's could be more descriptive. It's difficult to immediately tell what #final or #final2 is.
    • You also have some html and accessibility issues (there's a report of it on this page).

    You're on the right track. 😄

  • Vanessa Martin•155
    @vsm1996
    Submitted over 4 years ago

    Clock App Using React + CSS (Flexbox, Web-First Solution)

    3
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    Looks great! I love that you put the CSS files with the corresponding component -- before this I'd have it under ./src/css.

    The only thing I'd change is I'd use functional components.

  • AngelARVM•175
    @AngelARVM
    Submitted over 4 years ago

    Fylo data storage white bootstrap & Sass

    1
    Amir Jacobs•165
    @Amir-Jacobs
    Posted over 4 years ago

    I love your usage of variables in CSS. I also like that you've used prefixes where needed.

    Some things I'd do differently are:

    • Change width: x% to width: 100%; max-width: x%, as this helps with responsive design.
    • I'd limit the amount of classes I put on the html elements, because it becomes harder to read otherwise (though I understand that with bootstrap it's inevitable to add classes). I'd look into the BEM method.
    • I noticed you need to use margin-top: -340px; margin-right: -340px; on a class, which points to an issue in a part of the CSS.

    Overall it looks clean and structured. :)

View more comments
Frontend Mentor logo

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub