Skip to content
OverviewSolutions
12
Comments
5
Mahdy
@MhMahdi94

All comments

  • Vladimir•170
    @D0vl4Submitted almost 2 years ago

    Hello all! 🤠 I hope you are good and well! Any feedback is welcomed, thank you! 🙂

    Stats preview card

    1
    Mahdy•150
    @MhMahdi94Posted almost 2 years ago

    Hi, great work. i have small notice in your work, in responsive you use one value for screen width in media query,

    you have to use different screen sizes for mobile, tab ... etc

    feel free to ask me if you have questions

    0
  • Mahdy•150
    @MhMahdi94Posted almost 2 years ago

    great work, i like how you define the colors in style file. try to use flexbox or css grid

    1
  • Mahdy•150
    @MhMahdi94Posted almost 2 years ago

    Hi, Great Work ^_^

    first for icons you can find it in images folders in starter code

    for paragraphs, it better make it inside div and set fixed width, because it is effect on learn more button (you can check my solution)

    Feel free if you have any questions

    Marked as helpful
    1
  • Mahdy•150
    @MhMahdi94Posted almost 2 years ago

    Hi, your solution is great, but i have small notices

    1. you need to redesign the Learn More button
    2. use hover to add css effects to make button better
    3. also make your website responsive

    please feel free to ask me , you can check my code and then we can discuss

    0
  • Mahdy•150
    @MhMahdi94Posted almost 2 years ago
    1. Solution does not include semantic html
    2. accessible, need to improve layout and shadows
    3. look good on range of screen sizes
    4. well structured, readable and reuseable
    0