P
ApplePieGiraffe• 30,565
@ApplePieGiraffe
Posted
Hello there, adluders! 👋
Good effort on this challenge! 👏
A few things I suggest are,
- Making sure the navigation links in the footer of the page of the mobile layout are actually links by using the anchor tag to create them.
- I think closing any other open submenus in the header of the page when another submenu is opened would be a good idea (rather than having them all open at the same time).
- Adding some margin/padding around the paragraphs on the page so that there's some space between them and the edges of the screen in the mobile layout. 😉
Keep coding (and happy coding, too)! 😁
1
Adler G Luders• 1,580
@adluders
Posted
@ApplePieGiraffe thanks for the detailed feedback as always. And yeah I was being lazy 😂😂. After finishing the challenge I'm currently working on, I plan on refactoring this and three other projects that I submitted. Btw are you on slack? I would love to connect with you.
1
P
ApplePieGiraffe• 30,565
@ApplePieGiraffe
Posted
@adluders
No problem! 👍 Yeah, I'm on the Frontend Mentor Slack community—feel free to DM me there @ApplePieGiraffe! 😀
0
