using flexbox , js

Community feedback

• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted almost 5 years ago

  Hi, esraagamal! 👋

  Nice work on this challenge! I like the animation in your hamburger menu! 🙌

  In addition to emestabillo's feedback on your arrow buttons, I suggest,

  • Making the background of the arrows change to a lighter color (rather than changing their opacity) upon hover so that the content of the page doesn't show through the back of the buttons.
  • Also, there's quite a bit of space between the navigation links in the header of the page. Decreasing that space would bring them closer together and easier to see.

  Keep coding (and happy coding, too)! 😁

  12
• 

  

  Emmilie Estabillo•5,600

  @emestabillo

  Posted almost 5 years ago

  Hi Esraa, it's me again lol :-) Much improved wow, I can see a lot more control of the design. Just two things here:

  • It looks like you had a bit of a time aligning the arrows as the screen sizes change. One suggestion is nesting the arrows inside slide-pics (so that it becomes their parent), and then using position: relative on this image container. You would still need media queries because the layout is different on desktop, but it probably won't be as many as you have now.

  • The images start to look huge compared to the text between 900px to 1200px

  Hope this helps :-)

  12