Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Vladimir 170

    @D0vl4

    Submitted

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

    Mahdy 150

    @MhMahdi94

    Posted

    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

    @MhMahdi94

    Posted

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

    1
  • Mahdy 150

    @MhMahdi94

    Posted

    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

    @MhMahdi94

    Posted

    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

    @MhMahdi94

    Posted

    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