responsive landing page

Community feedback

• 

  P

  Serhii Orlenko•345

  @grifanoPosted 7 months ago

  Hi 👋 congratulations on your great job 👏 The code is very pleasant to read, components look good for each screen size.

  I have only suggested using jpg for exporting images like photos. You use SVG format for user avatars, and this format isn’t recommended for photos. It may not be obvious for this particular example, as you have a small image, however, if you export photos that will have a bigger size in SVG you get a huge image, which is not good for website performance.

  More practice with responsive layout makes you stronger in this field. I see you use min-width, which is great for your understanding of the Mobile First approach and this is great.

  You on the right path, this is a great job and keep going 💪

  0
• 

  dolapobj•310

  @dolapobjPosted 7 months ago

  Really great work. Reviewing your code and comparing it to my solution I learned two things.

  One is to use the <span> HTML element for decorating a piece of text inline, and also using media queries to specific styles to elements on the page when the viewport width is a certain width. I didn't do this for my solution and will begin using them from here on out.

  I don't really have any suggestions other than to perhaps consider using SASS/SCSS to build CSS files. I've been using it because it allows you to define global variables that will then be used in the rest of your CSS file.

  0