HTML5, CSS3, flexbox, mobile-first workflow

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello, Kiko! 👋

  Congratulations on finishing your first Frontend Mentor challenge! 🎉

  In addition to @EdwinSch has said earlier, I would recommend using the modern CSS reset by Piccalilli. It focuses on accessibility and is more modern than the one you currently used.

  One suggestion I have is to improve the alternative text. Alternative text should not be hyphenated (like code). Also, I suggest giving more information about the QR code. This way, the users will know exactly what the QR code is doing after they scan it.

  That's it! I hope you find this useful! 😊

  Marked as helpful

  2
• 

  P

  Ed•720

  @EdwinSchPosted over 2 years ago

  Hi! Congratulations on your first solution. :) About your question; there isn't really a right or wrong answer here as every project had it's own requirements. Building out a massive multipage application would have a very different approach compared to this single component. But in my overal experience: structure is the keyword. Start a project by thinking of, or sketching out, what your approach is going to be. Break down the design in a Component, Flex or Grid structure (whatever your preference) so you get a better view of how to build your layout. Logically structure any files that you will need or have recieved and last but not least; decide if you are going with mobile- or desktop first responsiveness.

  Happy coding!

  Marked as helpful

  2

  ---

  

  Kiko•20

  @Lushifer666Posted over 2 years ago

  Thanks @EdwinSch, this will be really helpful, I haven't thought of organizing that much before starting the project, will take note of this and try to implement this practice on the next project.

  0