All comments

• 

  oluwapelumi Fadiya•30

  @KodeNinjSubmitted over 3 years ago

  I will love your feedback please

  

  FYLO landing page

  2

  30
  

  Chris-Ann C.•300

  @cdc2021Posted over 3 years ago

  @oluwapelumi

  Great job on this challenge!

  Suggestions:

  • Have a read of your report as you have some accessiblility issues, maybe try adding a label for "email" which needs to match your input id rather than 'action'.
  • Ensure you are using the headings in chronological order and applying to sections.
  • The second page is very close to the first page so I would add some padding or margin to separate the elements.
  • The testimonial cards sits behind the access card, maybe have a look into your positioning.

  Hope this is useful.

  Happy coding :)

  2
• 

  Teri Eyenike•20

  @TerieyenikeSubmitted over 3 years ago

  Your positive feedback and criticism are highly welcomed.

  Do you like the overall look and feel of the Loopstudios landing page and how can I improve on the project if you found something missing?

  

  Mobile-first approach using CSS Grid and Flexbox, Loopstudios

  4

  30
  

  Chris-Ann C.•300

  @cdc2021Posted over 3 years ago

  Great job on this project! Looks and feels good. One thing your burger menu disappears when scrolling so maybe think about the colour scheme as it dissolves into the white background.

  Haappy coding !

  1
• 

  ETINOSA EKHATOR•220

  @Etinosa22Submitted over 3 years ago

  please guys rate my work? would appreciate your honest comments

  

  FOUR-CARD

  2

  10
  

  Chris-Ann C.•300

  @cdc2021Posted over 3 years ago

  Great work on this challenge, very responsive!

  Minor details, the contents within your header are quite close so maybe adding some padidng betweeen to space out the h2 and p tag. Also the opacity of the rely class should match the power class rather than the p tag.

  Hope this is useful.

  Happy coding :)

  0
• 

  Chris-Ann C.•300

  @cdc2021Submitted almost 4 years ago

  3rd project and loving this space :D

  Any feeback on responsiveness would be appreciated.

  

  Use of GRID and Flexbox to style

  2

  10
  

  Chris-Ann C.•300

  @cdc2021Posted almost 4 years ago

  Great, thank you :)

  0
• 

  Alejandro•10

  @Lerix78Submitted almost 4 years ago

  I can't figure out how to place the background images.

  Any suggestion?

  

  Profile card component

  2

  10
  

  Chris-Ann C.•300

  @cdc2021Posted almost 4 years ago

  Hi Alejandro,

  I would remove the background img from the HTML file and style in CSS using background-image: url(./); remove repeat - background-repeat: no repeat In order to position the images use tranform: tranlate (%, %) This should solve your background image issues and so make styling of the container easier.

  Hope this is useful. Chris-Ann

  1