Victor Crest - Responsive

Community feedback

• 

  P

  Remus D. Buhaianu•3,125

  @MiculinoPosted almost 3 years ago

  Congrats on completing the challenge, @PitagoraStrike!

  It looks really similar to the original design and you've successfully implemented responsive design.

  My only suggestion is to use more of the relative units such as rem, em, vh / vw, % instead of px.

  Marked as helpful

  0

  ---

  

  VicDev ☪︎•330

  @ViicDevPosted almost 3 years ago

  @Remus432 you're right that will make it more flexible, I'll work on it thanks

  I'm sorry I have a question I can't delete some designs and I already selected in the delete section, I would appreciate the help 🙏

  0
• 

  Web Wizard•5,690

  @rsrclabPosted almost 3 years ago

  Hi, @PitagoraStrike~ Contratulate on your solution to the challenge on FM platform. Really great work! Especially I like BEM structuring part of your work.

  The only thing I like to advice on this one is to add alternative text to image tags, so that there won't be any HTML semantic issues.

  Cheers~

  Marked as helpful

  0

  ---

  

  VicDev ☪︎•330

  @ViicDevPosted almost 3 years ago

  @tymren608 Thanks for taking the time to review the design, I'll work on it

  I'm sorry I have a question I can't delete some designs and I already selected in the delete section, I would appreciate the help 🙏

  0