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

HTML very Semantic, Mobile frist, grid layout and SASS

@MarlonPassos-git

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I'm very proud of my layout, but twist the fingers so that there is an error, so that I can improve it even more

Community feedback

P
Grace 27,950

@grace-snow

Posted

Hey

This looks good. You need to fix the html typo (article not artigle) mainly.

I can't look at css in this as you've not included an expanded (unminified version in the repo). Its good to include that every time in these challenges just so it's easier for people to review the code.

(Scss is fine to look at on this as there's not much there, but on bigger projects its a real pain to review code having to go in and out of lots of files)

1

@MarlonPassos-git

Posted

@grace-snow I didn't understand the part of the article and not the article, if you could give me more details...

Thanks for the feedback and the tip of not putting css minimized, I really hadn't stopped to think about it

0
P
Grace 27,950

@grace-snow

Posted

@MarlonPassos-git you have spelled wrong. It says artigle

Marked as helpful

1

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