Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
11
Comments
7
Omar
@omar-csdev

All comments

  • proabre•190
    @proabre
    Submitted almost 2 years ago

    news-letter-sign-up htms css and js

    1
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Hi, you are missing input validation upon submission. Try clicking on the button before typing a valid email in and you'll see what I mean. They also require you to show an error state for the user. Check the design images provided by frontendmentor.

    Also on smaller screen sizes you can scroll and see the confirmation message.

    Marked as helpful
  • Abdul Khaliq 🚀•72,380
    @0xabdulkhaliq
    Submitted almost 2 years ago

    NEWS HOMEPAGE 🎯 [ ACCESSIBLE - BEM - VANILLA CSS3 - ES6 ]

    #accessibility
    2
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    hi, I noticed that there are repsonsive issues on larger screens. Mainly the bottom half of the screen on full width 24 inch monitors.

  • Haiko•200
    @Haiko-E
    Submitted almost 2 years ago

    Newsletter sign-up form with success message

    #vue#tailwind-css
    1
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Hi, small tip: after submiting an invalid email you get an error hint which is good, but when you go ahead and try inputting a new email, the error hint stays there. If I didn't manage to make it clear, you can check out my submission on my profile from yesterday. Other than that looks good!

    Marked as helpful
  • Benjamin Lincoln•20
    @CuriousFullStacks
    Submitted almost 2 years ago

    Responsive product card using flexbox, rem units, and a media query

    2
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Hi, it looks good on large screens but not so much on smaller devices. I advice you to check that out and make it responsive using @media queries. Also on smaller devices the image should change to the mobile image provided in the files. You can achieve this using the <picture> tag, look that up it's not difficult. Good luck!

    Marked as helpful
  • void-404•10
    @void-404
    Submitted almost 2 years ago

    qr componend with html css

    1
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Well Done!

    Marked as helpful
  • Tom Kelly•40
    @tom-kelly9
    Submitted almost 2 years ago

    html, css, grid, flexbox

    2
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Hi, it looks fine on desktop view. But when you resize it towards 375px, it becomes clogged up sometimes. Go in dev tools and resize the window, you will notice between 700px width and 376px width that it's not as intended.

  • Ivan Brčina•70
    @Ivan-Brcina
    Submitted almost 2 years ago

    A simple first beginner Frontend Mentor project

    3
    Omar•240
    @omar-csdev
    Posted almost 2 years ago

    Very minor thing that I tend to notice: you can adjust the font size or the width of the card to match which words are at which line. What I mean is if you look at your version and see the last line you can see two words, while the solution shows three words.

    Apart from that, code seems good to me :)

    Marked as helpful

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

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