Skip to content
OverviewSolutions
12
Comments
15

Siddhesh Kamble

@Sk7867Mumbai350 points

Engineering student Eager to learn new things

Latest solutions

View more solutions

Latest comments

  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi ANDREW GRADY, There's a small problem with all the href in your HTML code.

    1. In your stylesheet link in head it should be ./style.css due to one missing . it is not linking to your stylesheet.

    2. Your svg icons are not in images folder on github so remove ./images/ just keep ./icons-sedans.svg or icons-sedans.svg in href.

    3. Add border-radius to your container or individual divs according to design.

    Happy Coding!!

    0
  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi Michelle, It looks good on desktop but in mobile view and tablet view, 3rd section is not visible, in body put min-height: 100vh instead of height: 100vh and padding-top: 20px or any value you like so it doesn't stick to the top in mobile view.

    Marked as helpful
    1
  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi LESTER NIEL FONG, Add <div class="overlay> inside img like this:

    <div class="card image">
      <div class="overlay"></div>
      </div>

    and give following properties to overlay:

    background-color: rgba(153, 53, 216, 0.6);
  height: 400px;
  border-radius: 10px 10px 0 0;

    Happy Coding!!

    Marked as helpful
    0
  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi JEFFREY IZELAAR, You need to change mobile breakpoint, keep it around 768px, 376 px is kind of wrong. 768px is tablet width. It looks good in all views.

    The font design seems similar.

    0
  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi KASHYAP PANDYA, Along with what @Thulani Gamteni suggested, add margin: 20px or any value to section in mobile and tablet view, that way your section won't take the whole screen in those views.

    0
  • Siddhesh Kamble•350
    @Sk7867Posted over 3 years ago

    Hi Bodgan, In your .main-div keep margin: 50px auto, it will give fixed top/bottom margin as 50px.

    For the first HTML validation error put <!DOCTYPE html> at the very start of your HTML code, for others I recommend removing the role attribute.

    Happy Coding!!

    Marked as helpful
    0
View more comments