Skip to content
OverviewSolutions
44
Comments
38
Michael Adebambo
@Blazing-Mike

All comments

  • Michael Adebambo•1,045
    @Blazing-MikePosted over 2 years ago

    This looks great! It is mobile responsive and the validation works well. If you want improve it, you can work on the password validation to accepts on >= 6 characters ( and combination of letters, numbers and symbols). The CSS looks great as well !

    1
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 2 years ago

    Great job you did here. Everything looks perfect, the only suggestion i have is the password input; it accepts an input of even two characters, in the password validation you can add that it returns an error for password <= 6 characters. If you want to go further you can add that it contains a combination of letters or symbols and numbers. Well-done Elizabeth

    Marked as helpful
    1
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Hmmn it's looks a bit distorted and I need to refactor the code

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Great attempt 🚀🚀. You should check if your body by any means has margin because there are little spaces around the page..... Did you use different images for desktop and mobile? You can use the picture element or use media queries to display different image of viewports

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    grande tentative que vous avez eu ... Pixel parfait et pour détailler. continuons à nous améliorer et à construire

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    On your mobile view. The image section before the footer should have four images displayed horizontally. You can use display grid or flex for that section and also use media queries to tweak any element you want to on different viewports

    LAYOUTS ( CSS GRID AND FLEXBOX) MEDIA QUERIES

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    On your mobile view. The image section before the footer should have four images displayed horizontally. You can use display grid or flex for that section and also use media queries to tweak any element you want to on different viewports

    LAYOUTS ( CSS GRID AND FLEXBOX) MEDIA QUERIES

    Marked as helpful
    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Noted. I was just focused on the countdown, I got tired of the UI. I would make changes right away

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Thanks. I would refactor it

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Everything works just fine 🌝. You could just align the container vertical to center on desktop mode.

    With . container { Display grid; Justify Content: center; Align items : center; Min-height:100vh }

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Great attempt you made. But the desktop site seem a bit distorted. You can check the height of the testimonial boxes and the spacing between the top container and bottom container

    Marked as helpful
    1
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Ok great attempt you had... Try aligning your card like give it some space top and bottom.. you can use margin tip and bottom or use

    Display: grid; Align-items: center; Min-height: 100vh; Justify-content: center;

    To align it vertically and horizontally

    Marked as helpful
    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Everything works fine 🌝... Great job.

    I tried reading through the code, it seems you used some ternary operators and some modern JavaScript syntax....

    1
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    wow... Everything to detail, great job man. Do you care to explain how I can make my tabbed section display flex... Also how I should make my navigation cover the logo and toggle bar. Thank you 👏

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    wow... Everything to detail, great job man. Do you care to explain how I can make my tabbed section display flex... Also how I should make my navigation cover the logo and toggle bar. Thank you 👏

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Nice one... You should check the margin side on Mobile screen... I'm currently doing this challenge too and it's hard to me... But I'm figuring out.

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    Beautiful 😍... Your code is working fine I love the Way you implemented sass even to this project.

    0
  • Michael Adebambo•1,045
    @Blazing-MikePosted over 3 years ago

    You did a great job on your first attempt.... You should increase the width of your boxes it's too small. Also use the rem or em units so as to make it Responsive.

    Marked as helpful
    1