Skip to content
Submitted over 3 years ago

HTML very Semantic, Mobile frist, grid layout and SASS

Marlon Passos•940
@MarlonPassos-git
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

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

  • Grace•31,270
    @grace-snowPosted over 3 years ago

    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
    Marlon Passos•940
    @MarlonPassos-gitPosted over 3 years ago

    @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
    Grace•31,270
    @grace-snowPosted over 3 years ago

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

    Marked as helpful
    1

Please log in to post a comment

Log in with GitHub

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