Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page - grid, flex, mobile-first

Camille 130

@fyrfli


Design comparison


SolutionDesign

Solution retrospective


A lot of the errors on the report don't make sense to me. Like ... I have no transforms in my code. Could that be coming from the fontawesome kit? Anybody know?

Community feedback

P
Grace 27,350

@grace-snow

Posted

Why are you adding in font awesome js? You don't need it

Might be good to get into the proactice of using svgs for your icons instead of importing a huge font for 3 tiny images

1

Camille 130

@fyrfli

Posted

@grace-snow Thanks. I don't much like using it but ... I dunno why I thought it was standard practice. I'll re-submit in a minute.

0
P
Grace 27,350

@grace-snow

Posted

@fyrfli you don't need to resubmit if that's what you meant. You can just push up new/updated code. You can generate a new report if needed

0
Camille 130

@fyrfli

Posted

@grace-snow That's what I did. I still have heading warnings but it's ok now. I won't be using fontawesome anymore.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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