Skip to content
OverviewSolutions
8
Comments
7
Khubaibul Islam Shakib
@khubaibul

All comments

  • Khubaibul Islam Shakib•310
    @khubaibul

    I have just checked your project. You have done it very well.

    Although you can improve some logic.

    First, the Side navbar not changing to summary when going to the summary section. Add-ons remain. Try to solve it.

    Secondly, in the summary section the total calculated amount is coming NaN (Not a Number).

    Finally, try to add a favicon.

    Best of luck.

  • Khubaibul Islam Shakib•310
    @khubaibul

    Hi! I have checked your project. You have done it very well. Although you can update some designs.

    First, please try to align the main content to the centre from the left. Or you can take more space for images and text+button content. Try to give an outline or border in the navigation link based on active status.

    There is one more issue that I have faced is, the number remains only one at the top of the shopping cart icon.

    Have a good day.

    Marked as helpful
  • Khubaibul Islam Shakib•310
    @khubaibul

    I have checked your project. You have done it very well. Keep it up.

    Marked as helpful
  • Khubaibul Islam Shakib•310
    @khubaibul

    Hi! I have checked your project. You have done it well. Try to improve some components like buttons, input field and selectable plan.

    First of all, try to align all buttons to be fixed at the bottom. Now your some button is under the div. Like Next Step, Back, Next etc. Secondly, use the input type "email" for an email input field.

    Then give some CSS effects based on the selected card. So that user can understand which card he selected.

    Also, your yearly plan isn't working somehow.

    Best of luck. Have a good day.

  • Khubaibul Islam Shakib•310
    @khubaibul

    Hi! I have checked your project. Very well done.

    I faced a problem while filling in the phone number field. Because the regular expression is a bit tricky. Try to use normal regEx for the phone number. It would be good for users I think.

  • Khubaibul Islam Shakib•310
    @khubaibul

    Hi. I have checked your project.

    The project went very well. You can still update some things if you want

    First, use the input type email for the Email Address and input type tel for the phone number. Then use some CSS for the selected plan so the user can understand which card he selects. You can use some CSS based on the checked attribute.

    You can also update in responsiveness. Input tags aren't fully responsive.

    Marked as helpful
  • Khubaibul Islam Shakib•310
    @khubaibul

    Try to change the input field text and input box background. Now the text is almost invisible. You can use focus for that.