Frontend Mentor

@AryaAnish121

Posted over 3 years ago

ok thank you but I didn't understood x icon do you mean favicon if yes it's the first solution I didn't know if the frontendmentor's same favicon that is mandatory. I will change it.

Please like it.

Thank you!

Kacper• 205

@gacbur

Posted over 3 years ago

@AryaAnish121 Yes, sorry, I meant favicon :) that's right. no problem, have a great day!

Arya Anish• 1,492

@AryaAnish121

Posted over 3 years ago

@gacbur thank you bye :)

please like

ApplePieGiraffe• 30,545

@ApplePieGiraffe

Posted over 3 years ago

Hey, Arya Anish! 👋

Congratulations upon completing your first Frontend Mentor challenge! 🎉 Good work on this one! 👍

The transition of the mobile menu icon is pretty cool! 😎

I suggest,

Giving the hero slider images a fixed height so that their height doesn't change and move some of the content around when changing slides.
Decreasing the space around the sides of the hero text to the left of the slider to give the heading and paragraph some more room (as in the original design).
Making the color of the mobile menu icon bars white so that they stand out against the slider images better and are easier to see.

Keep coding (and happy coding, too)! 😁

Arya Anish• 1,492

@AryaAnish121

Posted over 3 years ago

@ApplePieGiraffe thank you for the suggestion I am too new please help me:-

I gave them a fixed height but it always changes but in my computer and my smartphone it didn't change and about bars, I will do it, Thank you

ApplePieGiraffe• 30,545

@ApplePieGiraffe

Posted over 3 years ago

@AryaAnish121

After taking a look in Chrome's dev tools, you might be able to solve this issue by adding height: 495px (or something similar) to the hero image and getting rid of the padding around the text content to the right (use flexbox instead to center that text in its container). Adding object-fit: cover to the image may help keep them from distorting, too, when the page is resized. 😉

Keep learning and coding! 👍

Arya Anish• 1,492

@AryaAnish121

Posted over 3 years ago

@ApplePieGiraffe

Thanks for solution

Kacper• 205

@gacbur

Posted over 3 years ago

Hello Arya Anish, responsiveness is well done and overall you did a good job :)

one thing is only that the x icon on the smaller screens on mobile is coming a little to close to the links :) if you could make it a little more separated it would look even more awesome.

keep up the great work!

Arya Anish• 1,492

@AryaAnish121

Posted over 3 years ago

@gacbur

Thank You

Design comparison

Solution retrospective

Community feedback

Please log in to post a comment

Join our Discord community