Skip to content
OverviewSolutions
8
Comments
10
Sami Maachi
@Sami-21

All comments

  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great effort my friend. You should fix the background image and the selected plan container's height.

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great effort my friend. You should fix the background image and the selected plan container's height.

    0
  • ilian•60
    @iliand1Submitted about 3 years ago

    Any feedback is sought after.

    html scss

    1
    Sami Maachi•230
    @Sami-21Posted about 3 years ago

    The UI is clean great job, but it's not mobile friendly. I suggest optimizing it for mobile version.

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    the color of change can be changed by
    a{color:inherit;} ,this will let all a elements inherit the color. I hope this help you buddy.

    1
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    great job buddy most of the functionalities work perfectly. however, the design isn't optimized for large screens (1440px>) and small screens (550px<) . the theme transition is pretty decent but the border of the input field if flashing on theme switch. over all you did great managing the input ,keep up the good work.

    Marked as helpful
    1
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great efforts my friend. However, you need to get rid of the outline from input + button. it's would amazing if you try to mimic the design accurately, use the ressources provided by front end Mentor . keep up the good buddy.

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great job my friend. However, you should fix the background. Beside that everything is great. Good luck my friend 😊

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Excellent job buddy. The only thing you need to fix is the height of the container, so it's 100% like the design

    Marked as helpful
    1
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great job my friend. However, try to use the resources provided to you. Use the color codes for the button + text and the svg file for the background. Make sure it is align perfectly. Keep your head up and Good luck in your learning journey

    Marked as helpful
    1
  • lmef•15
    @LuisMiguelEstradaFernandezSubmitted about 3 years ago

    I am new to html css, if you could give me some advice it would be great. Thanks.

    HTML and CSS only

    2
    Sami Maachi•230
    @Sami-21Posted about 3 years ago

    great try my friend. try to fix the background by using background-repeat:no-repeat; in CSS. just keep on learing you are on the right path. good luck

    Marked as helpful
    1