Skip to content
OverviewSolutions
4
Comments
4
Tobaunta
@Tobaunta

All comments

  • Hamza Naeem•260
    @hamzafrontendSubmitted 3 months ago
    What specific areas of your project would you like help with?

    Open for constructive criticism.

    I would like to help me with border-radius on img on big screen, dont know it is looking weird, and secondly I dont know why when I add margin-block on main only top margin is applying but not the margin bottom, so for now I just applied padding-block which is working, but wants to know the reason.

    Responsive Recipe Page using HTML and Vanilla CSS

    1
    Tobaunta•40
    @TobauntaPosted 3 months ago

    all boxes are ticked, nothing to say but great job

    1
  • Tobaunta•40
    @TobauntaPosted 3 months ago

    I would set width to 100% and have a max-width instead of setting width to fixed. to be more responsive and look the same on all mobile devices.

    0
  • Tobaunta•40
    @TobauntaPosted 3 months ago

    Add a wrapper with full height to center the card vertical as well.

    2
  • Tobaunta•40
    @TobauntaPosted 3 months ago

    Looking at the slide it looks almost identical with the challenge but if you look on a device with big screen the card is not centered, this can be solved with a flexbox. On a device with smaller width there is no margin on the sides and the card cuts of from the left side.

    0