Skip to content
OverviewSolutions
12
Comments
6
Jhoell Opeyemi
@JhoellOpeyemi

All comments

  • Jhoell Opeyemi•250
    @JhoellOpeyemiPosted almost 3 years ago

    This is nice!

    Here's a suggestion though:

    • According to the design, the images should be above the text but yours are side by side on mobile ( I'm viewing from a 360px viewport ).
    0
  • Jhoell Opeyemi•250
    @JhoellOpeyemiPosted almost 3 years ago

    Really nice work!

    Just one suggestion though:

    There's an horizontal scroll on mobile view

    body {
     overflow-x: hidden;
}

    The code snippet above should fix it :)

    Marked as helpful
    0
  • Jhoell Opeyemi•250
    @JhoellOpeyemiPosted almost 3 years ago

    Great work! ⭐⭐⭐⭐⭐

    0
  • Jhoell Opeyemi•250
    @JhoellOpeyemiPosted almost 3 years ago

    Great work!

    You might want to remove the width you set on the card as the texts are breaking out in mobile view

    Marked as helpful
    0
  • duncanwhyte•140
    @duncanwhyteSubmitted almost 3 years ago

    Any feedback would be appreciated. Is the reverse margin an acceptable solution to the design of challenge ?

    I would like some input on how to organise my code to be neat and organised as I hope to get a job one day as a junior frontend developer.

    Any criticism will be well received.

    Blogr Landing Page SCSS,HTML and JAVASCIRPT

    3
    Jhoell Opeyemi•250
    @JhoellOpeyemiPosted almost 3 years ago

    Nice work Duncan Whyte! One issue though. There's an horizontal scroll on mobile screens, you could fix that by adding overflow-x: hidden; to the body. And you might want to have a look at your report

    0
  • Jhoell Opeyemi•250
    @JhoellOpeyemiPosted over 3 years ago

    This looks great!

    But between 1000px and 1150px screen sizes, the texts are busting out of their container (review div)

    Nice work though!

    0