Skip to content
OverviewSolutions
20
Comments
4
Gotyan
@eduardoramirezrojas

All comments

  • Natali 👻 Grimm•1,190
    @Grimm-N
    What are you most proud of, and what would you do differently next time?

    I'm excited to start the Junior stage. I've tweaked the color scheme a bit again, added some animation, and transition states for the Grid layout.

    What challenges did you encounter, and how did you overcome them?

    I would really like to make this grid more flexible and responsive, rather than just changing its state with media queries, but for now, this is what we have.

    What specific areas of your project would you like help with?

    I would love to hear suggestions on how to make the grid more flexible.

    👻 Grid Section: Bento (HTML + CSS + JS)

    #animation
    2
    Gotyan•390
    @eduardoramirezrojas

    This is a different level! 😲😳 I loved the background, it looks like it has a filter on it which makes it look even more interesting. The overall design is very well taken care of, it looks polished and visually appealing. However, I noticed that you are using a lot of div, you might consider using more semantic tags to improve the accessibility and SEO of your project.

    Other than that, it looks beautiful! 😊 I would love to get your feedback on my projects too; your perspective would be a great help in continuing to grow.

    Thanks and great work! 🙌!

  • Gotyan•390
    @eduardoramirezrojas

    Hi! I really liked your solution. It's clear that you put a lot of effort into it and there are many things to appreciate. However, I noticed that it could be improved. For example, when inspecting the layout and adjusting the screen size, some elements don't scale well on all devices. I did the same challenge a couple of days ago and I'd love to hear your feedback on my solution as well. Thank you very much!

  • Gotyan•390
    @eduardoramirezrojas

    Hi! I really liked your solution! However, I noticed a few areas where some adjustments could enhance the overall quality:

    Image Centering: The assets/images/illustration-sign-up-mobile.svg image is not centered on mobile devices. This affects the layout and user experience, so centering it would provide a more polished look.

    Use of <picture> Element: Instead of duplicating the <img> tag for responsive images, consider using the <picture> tag. This method is more efficient and can improve loading times. You can find more information about it here.

    Email Input Type: I noticed that you’re using a text input for the email field. Switching it to <input type="email"> would not only enhance form validation and improve accessibility but also allow the browser to autocomplete the user's email address, making it easier for them to fill out the form.

    Red Background: If the user enters an invalid email, consider using a setTimeout to make the red background disappear after a short period. This way, the user is alerted to the error without being distracted for too long.

    I’ve also created a similar solution, and I would love for you to take a look at my code and provide feedback. Overall, great work! With these adjustments, your project will be even more polished.

    Marked as helpful
  • P
    Abdirahim Ali Mahamoud•380
    @asad102
    What are you most proud of, and what would you do differently next time?

    solving js challenge is what makes my confident, I will solve more js challenge

    What challenges did you encounter, and how did you overcome them?

    in the desining parts adjusting the correct position, in the js part validating email while the user is writing instead of clicking submit btn as i do in my code.

    What specific areas of your project would you like help with?

    I am waiting for the developers feedback, thanks in advance

    designing and implementing newsletter form using tailwind & vanilla js

    #tailwind-css#jss
    1
    Gotyan•390
    @eduardoramirezrojas

    Nice, but it's not responsive design and doesn't match with the mentor frontend design