Responsive clipboard-landing-page solution using scss

Community feedback

• 

  walmanjm•425

  @walmanjmPosted over 3 years ago

  Hi, good morning! nice and responsive,

  if i may suggest would you please add some :hover to your social__links img

  in my case a friend suggest me to use : inline svg and the fill property

  but i cannot make it,
  instead i use : fontawesome

  its handle nicely with your social__links anchor tag

  thanks,
  happy learn, happy coding,

  Marked as helpful

  1

  ---

  

  ToNisamBioJa•330

  @ToNisamBioJaPosted over 3 years ago

  @walmanjm Thank you for the advice! Happy codding!

  0
• 

  Yashasvi Singh•235

  @Yashasvi2704Posted over 3 years ago

  Hi ToNisamBioJa, you have nicely implemented the solution. The implementation is just a bit off in terms of vertical seperation of items. Looks like you could use a design app(Adobe XD or FIGMA) to perfect your design to every pixel.

  An observation about the section to the right side of the monitor image. On a bigger screen(1440px + size ), the whole website responds to the change in screen size except for the Quick search, iCloudSync etc. section.

  Other than that, Great WORK.

  Marked as helpful

  1

  ---

  

  ToNisamBioJa•330

  @ToNisamBioJaPosted over 3 years ago

  @Yashasvi2704 Yes, you are absolutely right. Thank you for taking the time to look at the design and pointing these flaws out!

  1
• 

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  Your solution looks really good and responds well. 👍

  One small suggestion would be to set the alt text of the images that give no context to users to alt="" so assistive devices can ignore them.

  And also try to add some :focus state styles if you can, since it can improve keyboard user's experience.

  That's all from me. Have fun coding 👩‍💻

  Marked as helpful

  1

  ---

  

  ToNisamBioJa•330

  @ToNisamBioJaPosted over 3 years ago

  @FarisPalayi Thank you for the feedback!

  1