Skip to content
Submitted 7 months ago

Responsive Land Page Using CSS

Omar Ragabā€¢220
@Omar-Ragab-Projects
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social links profile
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I am proud of learning CSS.

What challenges did you encounter, and how did you overcome them?

Its funny, I delete this line when I start

then I take 1 hour in responsive problem to recheck that line again.

Community feedback

  • Daniel šŸ›øā€¢44,250
    @danielmrz-devPosted 7 months ago

    Hello @Omar-Ragab-Projects!

    Your solution looks great!

    šŸ“Œ It's recommended to use semantic HTML elements like <ul> and <li> for creating lists. This ensures that your code is more accessible, maintainable, and semantically meaningful.

    Here's and example on how you can refactor your code:

    After Refactoring

    <ul class="list-container">
    <li><a href="#">Github</a></li>
    <li><a href="#">Frontend Mentor</a></li>
    <li><a href="#">LinkedIn</a></li>
    ...
</ul>

    By using <ul> and <li>, you convey the structure of your content more clearly, making it easier for screen readers and search engines to understand. Additionally, it aligns with best practices for HTML semantics.

    I hope you find this helpful!

    Keep up the excellent work!

    Marked as helpful
    0
    Omar Ragabā€¢220
    @Omar-Ragab-ProjectsPosted 7 months ago

    Hi @danielmrz-dev,

    Thank you for this very helpful tips. :)

    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord