Skip to content
OverviewSolutions
6
Comments
4

Reckson Zirsangzela Khiangte

@Saga-sangaKeifang, Mizoram, India180 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • Reckson Zirsangzela Khiangte•180
    @Saga-sanga

    Hi Shalash, I've gone through your solution and regarding your question the I would recommend you try using <picture> html element for the SVG. https://www.w3schools.com/tags/tag_picture.asp

    Here's a link to my solution: https://saga-sanga.github.io/advice-generator/

    Marked as helpful
  • Reckson Zirsangzela Khiangte•180
    @Saga-sanga

    Hi, Toni congratulations on completing the project!

    I've gone through your solution and regarding your question on a dark background my solution was to have a blank div element in the header element. Set it to width: 100vw height:100vh and have a z-index lower than the actual navigation element.

    A link to my solution: https://www.frontendmentor.io/solutions/responsive-landing-page-with-navigation-jhF-XWlOHR

  • Reckson Zirsangzela Khiangte•180
    @Saga-sanga

    Hi, Anshul. Fist off congratulations on completing the project!

    I've gone through your project and a few pointers I would like to give you are:

    • Instead of using a fixed width in the .container class maybe try using max-width so as to make the card more responsive.
    • Instead of media="(min-width: 1440px)" change the break point to around media="(min-width: 640px)" so that the card adjusts itself only to smaller screens.
    • Also consider changing the font size, padding and margins so as to fit better into smaller screens.

    That is all from me. Thanks and happy coding!

    Marked as helpful
  • Reckson Zirsangzela Khiangte•180
    @Saga-sanga

    Hi Siddesh, congratulations on completing the challenge! 🚀

    Just a few points I've noted in your design:

    • Adjust the line-height of your h1 element so as to reflect the given design better.
    • Use some other tag than <article> for your image section so as to be more semantically correct.
    • The card takes up more than the view height on mobile view. You can fix this by adjusting the padding, font-size and margin of your text content.
    • Maybe utilize a mobile first design strategy next time.

    That's all I have to offer. I hope it helps. Thank you!

    Marked as helpful