Skip to content
Submitted over 4 years ago

Insure Landing Page

Emmilie Estabillo•5,580
@emestabillo
A solution to the Insure landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

This is my first time using inline svgs. I initially retained the fill property and discovered that hovering on spaces within the icon will not show its hover state. Only workaround I saw was to target the svg tag on the css file and declare initial and hover states for its fill property. Is this the standard?

Not sure what's going on with the screenshot...the live version doesn't show three lines of text in the CTA section in any screen width. I haven't coded in a while and I'd appreciate any comments for improvements or anything I've missed. Thank you!

EDIT: Took a new screenshot as updates to the platform were just pushed 🎉 Thanks Matt!

Code

Please log in to post a comment

Log in

Community feedback

  • Mark Mitchell•1,820
    @markup-mitchell

    Hi Emmilie,

    Inline SVGs can be targeted and styled with CSS just like HTML elements so you shouldn't think of the way you've done it as a workaround!

    Leaving properties like fill in the SVG will override the CSS in the same way as an inline style will override CSS, so yes, that's the way to do it. You might want to make the SVG width and height 100% and define those properties in the CSS class as well, just so you're not hard-coding those values into the markup.

  • Matt Studdert•13,611
    @mattstuddert

    Awesome work on this challenge, Em! Seems like your questions have already been answered, so I'll just say you've done a really good job! 🎉

    Great that the new screenshot is a more accurate representation of your solution. Feels great to have that update live on the site!

  • ApplePieGiraffe•30,525
    @ApplePieGiraffe

    Wow, amazing work, Emmilie Estabillo! 👏

    I really like your solution! 👍

    Happy coding! 😁

  • Dark-Lover•735
    @Dark-Lover

    hey,great job ;) . im working on the same challenge. the only issue that i didnt like in ur work is when u scroll while showing the nav-menu on mobile. i think u can fix that in just few lines, maybe a navbar position fixed.. something like that. good job ;)

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord