Skip to content
OverviewSolutions
7
Comments
8
Karim Ayman
@KarimAyman97

All comments

  • Karim Ayman•300
    @KarimAyman97

    Hi faridlakbar Great Job! I have some recommendations for you

    1- try to use separate file for CSS styling * for example * <head><link rel="stylesheet" href="styles.css" /></head> this will help improving code readability

    2- try to change <div class="card"> to a <main class="card"> and the <div class="attribution"> to a <footer class="attribution">

    this does not affect your project visually but it improves SEO as well as the accessibility of your project.

    I hope it helps!

    Other than that, great job!

  • Karim Ayman•300
    @KarimAyman97

    great Job ! but always try to Provide a more descriptive alt attribute for your image: <img src="assets/images/image-omelette.jpeg" alt="Delicious Omelette">

    Other than that, great job!

  • Karim Ayman•300
    @KarimAyman97

    Hello Umutcan Kocamış Great Job!

    I would like to suggest a solution. Begin by setting a transparent border for each button initially. Upon hover, apply the desired color and adjust the border accordingly

    
.sedans .learn-more {
        color: hsl(31, 77%, 52%);
        border: 1px solid transparent;  // give it initially transparent border
} 


.sedans .learn-more:hover {
  border-color: hsl(0, 0%, 95%);  // give it the desired colored border on hover 
  color: hsl(0, 0%, 95%);
  background-color: hsl(31, 77%, 52%);
}
    Marked as helpful
  • Karim Ayman•300
    @KarimAyman97

    Hi , Great Job! simple recommendation each <img> should have alt attribute which describes the image <img src="img/illustration-article.svg" alt="Illustration for the article" />

    this ensures accessibility and also improves overall SEO

    i hope it helps !

  • Karim Ayman•300
    @KarimAyman97

    hi revanthv999 Great Job !

    1- for your question Here are some best practices to arrange components vertically

    a) Use block-level elements such as <div>, <section>, <article>, or <p>

    b) Use the CSS Flexbox layout for vertical alignment. The flex-direction: column;

    c) CSS Grid Layout . Set the grid container's grid-template-rows

    2- I have some accessibility and semantics recommendations for you.

    try to change <div class="container"> to a <main class="container"> also each image should have alt attribute which describes the image <img class="profile_photo" src="./assets/images/avatar-jessica.jpeg" alt='avatar-jessica'>

    this does not affect your project visually but it improves SEO optimization as well as the accessibility of your project.

    I hope it helps!

    Other than that, great job!

    Marked as helpful
  • Karim Ayman•300
    @KarimAyman97

    hi Andrew Teece great job !

    i have a simple comment that every <img> should have descriptive so <img src="assets/images/avatar-jessica.jpeg" class="card--img" alt="" /> should be <img src="assets/images/avatar-jessica.jpeg" class="card--img" alt="avatar-jessica" />

    I hope it helps!

    Other than that, great job!

    Marked as helpful
  • Karim Ayman•300
    @KarimAyman97

    hi Shivam singh gautam great work . i have some recommendations for you

    1- try to use separate file for CSS styling * for example * <head><link rel="stylesheet" href="styles.css" /></head>this will help improving code readability

    2- Add descriptive alt attributes for your images to improve accessibility. <img src="assets/images/logo.svg" alt="Frontend Mentor Logo" class="logo">

    3- Make sure your navigation links have actual URLs. Currently, your <a> tags in the navigation are empty. <li><a href="#">Home</a></li>

    I hope it helps!

    Other than that, great job!

    Marked as helpful
  • Ciprian Danila•110
    @cipdanila
    What are you most proud of, and what would you do differently next time?

    Improvements made to the project based on HTML5 semantics, accessibility, and BEM.

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

    It wasn't really that difficult, so I wouldn't say I encountered too many difficulties.

    What specific areas of your project would you like help with?

    I would try a different approach to achieve something of higher quality. It seems it didn't quite turn out as intended.

    Social links profile (html & css only)

    2
    Karim Ayman•300
    @KarimAyman97

    hi Ciprian Danila great work you did. I have some accessibility and semantics recommendations for you.

    1- try to change <div class="container"> to a <main class="container"> and the <div class="attribution"> to a <footer class="attribution">

    this does not affect your project visually but it improves SEO optimization as well as the accessibility of your project.

    I hope it helps!

    Other than that, great job!

    Marked as helpful