Skip to content
OverviewSolutions
31
Comments
11
Haybuka Paschal
@Haybuka

All comments

  • Haybuka Paschal•670
    @Haybuka

    Hi... Hope this helps (thumbs up if it does). Easiest way to handle error checks is to make use of the onsubmit event attached to the form.

    Also, your name field, try using a fixed length for name, or truncate it when you are about to display.Using a long name broke your card UI, and alerts as error display is a bad UX.

    The keypress event might work, but it is not the best way to go about it.

    Also use the proper semantics for your forms. There is a button element, no need to convert an input tag to a button.

    Using the onSubmit event with an error check library like formik, goes a long way to make your work easier, and checks simpler.

  • Haybuka Paschal•670
    @Haybuka

    wow, nice... Good to know there is a functional library for drag and drop.. I Had to adapt one using vanilla js (my code), most i saw were not compatible with react... good job chief

  • Haybuka Paschal•670
    @Haybuka

    For the data collation, i am not sure how i wired mine, didnt really cross check before submitting, but you can wire it all to be a form, that way you can do a validation on submit which just checks the input fields and type

  • Haybuka Paschal•670
    @Haybuka

    hi,good job.. you could try modellling the result you want,then recreate that using document.createElement (its best to save it as a function so its reusable). This create element will then build your cart data whenever add to cart is clicked. to make validation easy,the data on the product right can be made as a form, that way you can easily validate before you send to cart. I hope you understand, would be better explaining via a voice note i guess... good work thouigh

    Marked as helpful
  • Haybuka Paschal•670
    @Haybuka

    Hi.. Maybe take a look at this tutorial, they sure did help me CSS : https://www.youtube.com/watch?v=I9XRrlOOazo&list=PL4cUxeGkcC9gQeDH6xYhmO-db2mhoTSrT CSS-POSITION : https://www.youtube.com/playlist?list=PL4cUxeGkcC9hudKGi5o5UiWuTAGbxiLTh CSS-FLEXBOS : https://www.youtube.com/watch?v=Y8zMYaD1bz0&list=PL4cUxeGkcC9i3FXJSUfmsNOx8E7u6UuhG

    Good job, keep it up.

    Marked as helpful
  • Haybuka Paschal•670
    @Haybuka

    Oh thanks for the feedback.. normally I pay attention to these points you've made while coding (and I also use Chrome light house for audit),but for this project my focus was mainly on the use of tailwindcss, so I allowed myself that liberty. ... Thank you,really appreciate that you took the time.

  • Haybuka Paschal•670
    @Haybuka

    Tight... Good job <3

  • Haybuka Paschal•670
    @Haybuka

    Hi... Nice Usage of JQuery, Maybe try use a separate JS file instead of the embedded with HTML... For the local storage, i couldnt find where you set your data to localStorage,you have to set it up,before you can pull from it.... Nice work on the documentation also

  • Haybuka Paschal•670
    @Haybuka

    Nice work on the code documentation

  • Haybuka Paschal•670
    @Haybuka

    thank you for this @Roman, will make sure to apply it when next i come across such.

  • Haybuka Paschal•670
    @Haybuka

    Thanks it did help...i'd love to see your finished work when its done,i hope to clarify some doubts i have concerning positioning... Thank you