Skip to content
OverviewSolutions
17
Comments
12
Mustafa
@Mostafa-T

All comments

  • Mustafa•415
    @Mostafa-T

    Its perfect !! 🔥 there is a little adjustment for a better look which is the box-shadow of the main container could have more blur

    Happy coding 👨🏾‍💻

    Marked as helpful
  • Mustafa•415
    @Mostafa-T

    Hey 👋 ,

    Everything looks perfect 🔥 but I just you should have a look at the compatibility of the css code.

    Happy coding 👨🏾‍💻

  • Mustafa•415
    @Mostafa-T

    Hey champ 👋,

    Its really nice try with your project.. there are actually many css problems with it

    • first it will be better if the main container is centered to the middle of the screen. You could actually check the flex box for that
      display: flex; align-items: center; justify-content: center

    • check the fonts because its not working

    • check the padding to for better look

    • you should try another way to make the line for example check the ::after pseudo element

    Keep learning and practicing 🔥

    Happy coding 👨🏾‍💻

  • Mustafa•415
    @Mostafa-T

    Hey 👋, The design is really perfect, you are rocking it 🔥

    There is only one thing that you might forgot which is the primary button box-shadow, it will be better if you considered this in your next project adjustment.

    happy coding 👨🏽‍💻

    Marked as helpful
  • Mustafa•415
    @Mostafa-T

    Hey, 👋
    Your design is really good and responsive too, Keep rocking 🚀 there are some stuff I hope you consider when adjusting this project:

    • It will better if you adjust the size of the fonts.
    • the paragraph in the original design is a bit lighter.
    • the box shadow of the main container should have more radius and blur for example: /* offset-x | offset-y | blur-radius | spread-radius | color */ box-shadow: 0px 0px 5px 3px rgba(0, 0, 0, 0.2);

    Happy coding 👨🏽‍💻

    Marked as helpful
  • Mustafa•415
    @Mostafa-T

    Thank you so much for you feedback ! I will be working on improving it more soon 🤓

  • Mustafa•415
    @Mostafa-T

    Thank you so much!! Really I can't be more grateful and for sure will improve this project. ❤️❤️

  • Mustafa•415
    @Mostafa-T

    Hey, I was reviewing your project and i found a problem here with the responsiveness. The subscribe button of the news letter is floating to the left.

  • Mustafa•415
    @Mostafa-T

    First of all thank you for reviewing my code 😁. I really appreciate your improvement suggestions and will update my projects with your suggestion.

  • Mustafa•415
    @Mostafa-T

    Thank you !!

  • Mustafa•415
    @Mostafa-T

    Thank you !!

  • Mustafa•415
    @Mostafa-T

    Hey Victor,

    Thank you for your feedback !! Yeah actually i was trying to add a ripple effect to the buttons, but it didn't go well so I will add hover state and animation on these buttons later on