Skip to content
OverviewSolutions
7
Comments
4
Valeh Ismayilov
@vlhsmylv

All comments

  • Muhammadjavohir•280
    @MuhammadjewelSubmitted about 1 year ago

    15th project is ready 🎉

    The projects includes email validation and its submission to Google Sheets. After the successful saving, page saves the email in localStorage and shows the success message on next visits.

    I think I should refactor some JS-related code. Also, error message for email submission (if any) could be shown. Light theme addition is also an option.

    Any constructive feedback is welcome.

    Pod request access page w/ email validation (saved to Google Sheets)

    1
    Valeh Ismayilov•140
    @vlhsmylvPosted about 1 year ago

    Hello Muhammadjavohir!

    Looks great!

    The only thing you may edit in design is height. You may fit the container in one page. As I spotted so far there is an unnecessary scroll.

    👐 Good luck in your development!

    0
  • Iwan Kurniawan•50
    @iwan-14Submitted about 1 year ago

    I just completed my second challenge of creating responsive product preview card. Happy to hear suggestions on methods of getting more accurate on this. Thanks.

    Product preview card

    1
    Valeh Ismayilov•140
    @vlhsmylvPosted about 1 year ago

    Hello Iwan Kurniawan!

    Looks good so far, but there are some points I can mention for your development:

    • Change the background of the body, so the component will be more visible.
    • Change font-weight of "Add to card", so it will be more effective and look better.
    • Make the "PERFUME" text smaller.
    • Make the title smaller.
    • Add line-spacing to the paragraph below the title.
    • Remove the top margin for the mobile version.

    👐 Good luck in your development!

    Marked as helpful
    1
  • Valeh Ismayilov•140
    @vlhsmylvPosted about 1 year ago

    Hello ManiNasih!

    Looks great so far.

    • You may add a gap between your main container and attribution.
    • You may make $169.99 text smaller.

    👐 Good luck in your development!

    1
  • Valeh Ismayilov•140
    @vlhsmylvPosted about 1 year ago

    Hello Gaurav Verma!

    There are some issues I would to mention for your development.

    • Try to remove the margin from the body, so you won't have white spacing in the body container.
    • Add hover effects to the form button also, I saw you did this in the Success Message component and removed the outline on hover.
    • If the email is not valid then change the background colour and text colour of the input field and remove the outline from the input (blue outline).
    • Add spacing between texts in Success Message.
    • You may add texts in the given design to your solution.
    • Try to reduce the gap between lists in the Form component.
    • You may think about changing sizes and some colours, for example, the button size should be bigger and the background colour should be dark-slate.
    • For fun, you can show inputted email in the Success Message component.

    👐 Good luck in your development!

    0