Sass, Flexbox

Community feedback

• 

  Arindam Sankar Das•90

  @ArindamSankarDasPosted over 3 years ago

  Hey buddy, I looked up at your source code on Github and the first thing I noticed was that your project structure is a complete mess. Try to structure it properly next time so that other developers do not have to spend much time looking for the specific file.

  Now, from the code basis :

  You could've have used separate files for the variables, colors and repeated block of codes to make your root SCSS file, i.e. the styles.scss file a little shorter.

  Secondly, if you're new to frontend then it's completely fine to use the current html pattern but try to use less html tags and utilize pseudo elements that CSS and CSS3 provides us with.

  Other than that keep trying, keep practicing and you'll get better at it.

  1
• 

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  Remove the / from the image path to get it working👍 It would look like this: images/image-victor.jpg

  0