Easybank Landing Page

Community feedback

• 

  calebmcmains•170

  @calebmcmainsPosted over 1 year ago

  Hey Karam! Nice work. I can tell you spent a lot of time to make sure that things looked good at both the 1440 screen size and the 375 screen size. I noticed that there was't much time spent on the in between screen sizes reducing how responsive of a landing page it really is.

  Something I'd say that could be an area of improvement would be the use of Flexbox and/or Grid. There are so wonderful CSS properties in those tools that make spacing and centering html element really easy. When I looked closer at your code you used a lot of margins, gaps and padding to make it look visually correct but in the end was probably tedious to do and doesn't help with the need for responsiveness.

  Hope this is helpful as you continue to grow as a frontend developer!

  Marked as helpful

  0

  ---

  

  Karam Singh•1,050

  @karam-7Posted 10 months ago

  @calebmcmains thank for your advice and sorry for late reply. I'm new in this field so i do so many mistakes and thanks again

  0