Skip to content
OverviewSolutions
12
Comments
25

Precious Nkwor

@Jeen-Presh440 points

Hi there, Hookage Web Development .. Support My Work BMC link https://bmc.link/phenoixuchx

Latest solutions

View more solutions

Latest comments

  • Precious Nkwor•440
    @Jeen-Presh

    Hello melvin

    • Please I would like to know how you knew the exact bg colors for the card background, is there any resource you can recommend for me to read?
  • Precious Nkwor•440
    @Jeen-Presh

    Hello melvin

    • Nice solution, I'm currently learning javascript now, so I decided to do a light project to practice dom manipulation.

    • Also I love your solution, and I'm using it to learn as well by breaking it down which helps me to learn more about accessibility, I noticed you also use sass and the 7-architecture system which is pretty cool. I love sass also.

    • The only thing I wanted to mention is the use of import in sass, I was using them also until I found out it has been deprecated, and instead we now have to make use of @use and @forward.

    if you are interested in learning @use and @forward you can use this link. Kevin powell.

    Have fun

    Marked as helpful
  • Precious Nkwor•440
    @Jeen-Presh

    Hello Shane, congratulations on completing your fem challenge

    • Always try checking your fem report section for errors, can help you figure out where to correct your project.

    • Try warping the code below in a header semantic element.

    
    <h1><span>Reliable, efficient delivery </span> Powered by Technology</h1>
    <p class="sub-text">Our Artificial Intelligence powered tools use millions of project data points to ensure that your project is successful</p>

    Hope you find this helpful. Happy coding

    Marked as helpful
  • Precious Nkwor•440
    @Jeen-Presh

    Hello favour, congratulations on completing your fem challenge.

    Some suggestions to help improve your project

    • Always ensure to wrap your main content with the main element for accessibility and semantic HTML markup

    more info here MDN main link

    • The code below should be wrapped with an h1 tag
     <h3 class="heading">
        Improve your front-end <br> skiils by building projects
      </h3>
    • Give your img element an alt tag. alt tags are good for accessibility for screen-readers

    Hope you find this helpful, congratulations once again. cheers

  • Precious Nkwor•440
    @Jeen-Presh

    Hello Kariath, congratulations on completing your first fem challenge.

    Some suggestions to help improve your project

    • Always ensure to wrap your main content with the main element for accessibility and semantic HTML markup

    more info here MDN Link Main Element

    • The code below should be wrapped with an h1 tag
    <P>Improve your front-end skills by building projects</P>
    • Give your img element an alt tag. alt tags are good for accessibility for screen-readers

    Hope you find this helpful, congratulations once again. cheers

  • Precious Nkwor•440
    @Jeen-Presh

    Hello George, congratulations on competing your fem challenge

    Some suggestions to improve your project

    • You should always wrap your main content with the main element.

    more info MDN main element

    • Make use of the picture element, since the challenge has two different images for different screen sizes.

    more info MDN link picture element

    -Make use of the del element to wrap the old price, add a span tag with a class of sr-only, and inside should have a text of the previous price. then use CSS to make the span element hidden, this makes it easier for screen-readers to pick the information properly.

    more info MDN del element

    more info sr-only class

    • The only heading should be perfume.

    Happy coding. I hope you found this helpful.

View more comments