Skip to content
OverviewSolutions
4
Comments
3
MiloOliveira
@MiloOliveira

All comments

  • MiloOliveira•70
    @MiloOliveira

    for not occuping all the screen, you can use the scale command in css. I reccomend you to see if can re-order the div class, in that way you use less lines. for the quotion mark, you can use z-index for put it behind and positon absolute for making her not coming out of the block. instead of doing div1, div2... you can do div one, div two... this way, you put two classes instead of one, and you can generate rules for all of the divs in a single time, and if someone is specific, you use only .one or .div.one

    Marked as helpful
  • P
    librart•150
    @librart
    What are you most proud of, and what would you do differently next time?

    I still need to learn and practice more about flexbox, grid, and media query for responsive web.

    What challenges did you encounter, and how did you overcome them?

    I still need to learn and practice more about flexbox, grid, and media query for responsive web.

    What specific areas of your project would you like help with?

    I think I would love to get feedback for all of my code from HTML structure to CSS styles, especially the flexbox & grid layout and the responsive design.

    Four card feature section using HTML CSS Flexbox Grid Media Query

    1
    MiloOliveira•70
    @MiloOliveira

    in style.css you can eliminate elements that you're not using, the less code lines, the better

  • MiloOliveira•70
    @MiloOliveira

    The alternate text for important images, like the product, it's better have a real description, for the person know what is. The site works well in a lot of different resolutions and gives a similar result to expected. The button don't have an indicator that is clickable, except for his color, would be good put a change to the cursor, using <cursor> and even an animation, if you want.