All comments

• 

  Kozharina•240

  @KozharinaSubmitted over 1 year ago

  

  Multi-step form (HTML, CSS, JS)

  1

  10
  

  georgeokez•80

  @georgeokezPosted over 1 year ago

  Great solution, one can tell that you've really put in a lot of work to get this done. Kudos 👏🏻!!

  I just have a few things I'll like to point out.

  1. I think you can remove the number selection on the sidebar when the user moves to the next step and only maintain the current step.
  2. I noticed that when the user toggles to the yearly plan, the labels on the addons and summary step don't change accordingly. That's something you might want to fix. A Suggestion would be to modify the labels for steps 3 & 4 within the click event listener for your switcher component here

  Other than that, the solution looks good. 👍

  0
• 

  georgeokez•80

  @georgeokezSubmitted about 3 years ago

  Really excited about frontend, being a backend engineer this is really fairly new for me. Would appreciate any feedback about the code.

  

  Order Summary Component

  2

  10
  

  georgeokez•80

  @georgeokezPosted about 3 years ago

  Thanks for the feedback, I'll keep all of what you've said in mind when starting out new projects.

  0