Fylo landing page two column layout - HTML&CSS - using flexbox

Community feedback

• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted about 3 years ago

  Hey, This is really great work! I like it. So to answer your question, simply use the CSS pseudocode property (.classname:hover{code}). If you want to add borders and all those other changes, it will work. So, all I can say is;

  1. For the responsiveness, I think you needed to apply the mobile style at a width around 450px. Cause the rest of the stuff would have fit in other sizes but It still like that it is responsive!
  2. I noticed that you added some margins to the containers of the other sections like (".about-fylo") and I think it would be better if you put the margins on the content so that you avoid the whitespace at the right and left of those containers.
  3. To solve the HTML and accessibility issues, I think you should use classes to avoid duplicate ID's. Otherwise, I liked the solution, it is really awesome.

  Marked as helpful

  0