Skip to content
OverviewSolutions
32
Comments
32

Emmanuel Akpan

@heisemmaco-devLagos, Nigeria790 points

Tech enthusiast with a passion for solutions. Always willing to collaborate. Git/GitHub, HTML, CSS & JavaScript.

I’m currently learning...

Currently learning JavaScript

Latest solutions

View more solutions

Latest comments

  • Emmanuel Akpan•790
    @heisemmaco-dev

    Hey Ega, you did i really good job there with design and also with javascript but there is one thing that you didn't get well

    The issue lies in the if statement checking if the email input is empty. Currently, it uses email.value === "" to verify emptiness. However, this condition doesn't account for spaces, so if spaces are entered, it still registers as content in the input.

    To solve this, consider using email.value.trim() for the condition. This adjustment checks if there's no content after trimming spaces. So, the updated code would look like this: if (email.value.trim() === "") { email.style.outlineColor = "red"; errorMsg.textContent = "Email cannot be empty"; } Now, it'll properly identify an empty input, including cases where there might be only spaces. This way, the form validation becomes more accurate.

    I hope your fine this helpful

    Marked as helpful
  • Emmanuel Akpan•790
    @heisemmaco-dev

    Hi bro, you did a very good job in your coding challenge, i admire the way you structure and style the web page, but you missed only one thing which is the background color, i can see that you mistakenly use container background color as the main background, that wrong, you can fine the color description in file named style guild, if you fine this comment helpful you can mark it a helpful and follow me, thank

  • Emmanuel Akpan•790
    @heisemmaco-dev

    wow!! bro you try oo, e no easy abeg, continue your good work and looking forward to see you next project bro

  • Emmanuel Akpan•790
    @heisemmaco-dev

    Hi Yeiifrensk, Great work right their, I love the responsiveness of the card but on Mobil and desktop but their is one thing you you missed the background color, the background color is not supposed to be with, you can fine the background color on the style guide.

    I hope you fine this helpful

    Marked as helpful
  • Emmanuel Akpan•790
    @heisemmaco-dev

    Hello there 👋. Congratulations on successfully completing the challenge! 🎉

    I must say you did a wonder job their, main look that the solution it look as if you use css flame works but you did it with pure css nice work their, i really love layout.

    continue the good works bro, no cap, you did a really things their,

    i hope you find this comment more encouraging and helpful

    Marked as helpful
  • Emmanuel Akpan•790
    @heisemmaco-dev

    Hello there 👋. Congratulations on successfully completing the challenge! 🎉

    I have other recommendations regarding your code that I believe will be of great interest to you. CSS: I WILL LIKE TO GO STRAIGHT TO THE POINT your images look more of white you can correct this by making the background color of the image div purple and then apply this property to the div: "background-blend-mode: multiple;"

    secondly, your image shrink with the whole card, i took a look that your code an saw that your use the source tag to set you image media quire, that good acually but you have to use only one instead of setting two source code at your picture tag

    I hope you find this helpful 😄 Above all, the solution you submitted is great !

    Marked as helpful
View more comments