Skip to content
Submitted over 3 years ago

Responsive Design using the CSS Flexbox and the good'ol Vanilla CSS

Dan Veranga•40
@danveranga
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Is it possible for any of experienced front-end developer out there to give me some recommendations and useful advise on how to make my codes more neater, organized and more professional?

Code

Please log in to post a comment

Log in

Community feedback

  • Chamu•13,860
    @ChamuMutezva

    Hi.

    • consider elements that give your site meaning. Elements such as main , header, footer are more important than divs.
    • the first heading element of a site has to be an h1, and the headings has to flow in order without skipping any headings
    • you can take another screenshot to reflect changes that you may have made.If you are not a Pro member - screenshots are limited
    Marked as helpful
  • Mohammed Jelidi•170
    @MedJelidi

    I'm by no means an experienced front-end developer but I can share with you some tips:

    • If you have a value that is repeated multiple times in your styles (colors in your case), put it in a variable and call it every time you need it. Besides, whenever you want to change that color, you only change that variable's value instead of going through all your code.
    • Avoid using uppercase when naming selectors.
    • For formatting your code, I recommend Prettier extension on VSCode if you're not using it already.

    Other than that, your code looks clean already.

    Marked as helpful

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