Article Preview - 8/10 🥳

Community feedback

• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi Sammy,

  Great job finishing this project! It works perfectly for both desktop and mobile devices! You even implemented BEM methodology :) The next technology you might want to learn is SASS, Bootstrap, or TailwindCSS.

  Keep it up :)

  Marked as helpful

  1

  ---

  

  Sammy•280

  @soyedPosted about 3 years ago

  @fidellim Thanks for the feedback! Yes, I am learning SASS at the moment. Also, I use tailwindcss a lot but for the challenges that don't require a js framework, I am sticking to just plain CSS that's why.

  1

  

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  @soyed Yes, that is great! Keep it up :)

  0
• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted about 3 years ago

  Hey, Sammy, I would like to correct your ratings to 9/10😁 The 10 will come after you add a level one heading tag to your html most preferably on the bold title preceding the paragraph. Just wrap it in an h1 tag within your div <div class="article--body__title"> But honestly this is a 10/10. I like the pop up that links to the social media platforms when the mouse hovers over the share button keep up the good work and keep on coding.

  Marked as helpful

  1

  ---

  

  Sammy•280

  @soyedPosted about 3 years ago

  @tesla-ambassador Thanks for the feedback!! I will make the HTML change as well!!

  0