@NikolaD93
@NikolaD93
Submitted
Hello everyone 👋
It's been a while since I submited a solution here, so I did an easy challenge just to stay in shape.😁
Had a problem with footer images, so I didn't use them.
Feel free to leave a comment about the code, so I can improve it!
Cheers! 🎉
@NikolaD93
Posted
Almost forgot 😁 , I also made a video preview of the project on my YouTube channel, check it out 👇
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
👋 Well done my friend. Looks amazing! 🌟
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! 🙌
I have a suggestion for you:
Put all your footer social icons in an unordered list - <ul></ul> tag, then in list - <li></li> tags, which then have <a></a> tags. 👇
https://www.w3schools.com/html/html_lists_unordered.asp
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Well done my friend, just keep on practicing and improving your skills. 👌👌
Try to do some harder challenges, you've mastered newbie 😁😁✌
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Hello my friend! 👋
Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@klassno3
Submitted
I'm not sure whether my method for making it responsive was the appropriate one because this was my first project with Tailwind CSS. Any comments would be greatly appreciated.
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! 🙌
I have a suggestion for you:
1.Make sure to write some description on all of your images in the alt attribute. 👇
https://www.w3schools.com/tags/att_img_alt.asp
2.You can put all of your element in one parent container with a <main></main> tag.That represents the use of semantic HTML, which is more descriptive and SEO friendly.
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@akash4102
Submitted
how to make this more better?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! 🙌
I have a suggestion for you:
1.You can do some reset CSS styles. By resetting your styles, you avoid defaulting to the browser’s built-in styles, which differs from browser to browser. 👇
You can reset styles like this, just write the following code in the CSS file:
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@Abdo-al-R
Submitted
Hello this is my solution for this challenge , any suggestions ?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! 🙌
I have a suggestion for you:
1.Make sure to write some description on all of your images in the alt attribute. 👇
https://www.w3schools.com/tags/att_img_alt.asp
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@tizerk
Submitted
Any feedback/thoughts are greatly appreciated!
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! Everything looks great and the code is clean. 🙌
One tiny suggestion, you can put all of your element in one parent container with a <main></main> tag, instead of putting it in a div .design-container. That represents the use of semantic HTML, which is more descriptive and SEO friendly.
Hope this was helpful for you!
Happy holidays! 🎉✌
Marked as helpful
@BuluBerry
Submitted
Did I use the best solution to keep the QR container in the center of the screen?
@NikolaD93
Posted
Hello there! 👋
Congrats for completing the challenge! 🙌
Yes your solution is good well done, I have a suggestion 👇
To center the container use display:flex; justify-content:center; align-items:center.
Here are some other ways to center a div or text 👇
Happy holidays! 🎉✌
