All comments

• 

  Alphonzo Escolar•120

  @phonzdevSubmitted almost 4 years ago

  I think my code is messy. I just focused on making it work. Please tell me how I can improve! :) Thanks!

  

  Mobile-first solution using Flexbox

  1

  10
  

  Oscar Alejandro Lopez Ramos•80

  @MachochinLopezPosted almost 4 years ago

  Desktop looks perfect, the only thing is that the background image at the bottom does not stay at the bottom if the screen is bigger. It's like fixed at a specific position. Mobile also looks nice on the target breakpoint. The problem is when you resize the window a little bit, it starts getting a little messy.

  Everything between the desktop breakpoint and the mobile breakpoint has some weird behaviours, mostly paddings I think, but those are some easy fixes you can do using your media queries maybe :) Also the top desktop image of the background doesn't change to the mobile version while the bottom one does. I think those are the main issues I saw with it. Also, the report says you have 6 html issues, you can also check those to see what other things the FEM site tells you.

  Nice job c:

  2