Results summary component with flex-and semantic HTML

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted 9 months ago

  Hi there

  This is overflowing my screen because you've used explicit widths throughout. Take these principles I share and I recommend you pause and refactor all earlier projects applying the same before moving on

  1. You don't need a section to wrap articles, they are distinct. Personally I wouldnt use articles for this as there is nothing to be gained from it and this content would never be syndicated across different sites/pages. If you only are separationg content for layout purposes, divs are fine.
  2. The scores like 76/100 are one paragraph not two.
  3. Decorative images like these icons should have empty alt.
  4. The b tag is deprecated. Did you mean to use strong?
  5. The attribution should be in a footer landmark outside of main.
  6. It's better for performance to link fonts in the html head instead of css imports.
  7. Don't use any heights or widths except for the icons and the circle around the overall score.
  8. Always use a modern css reset at the start of the styles in every project. Get into the habit now. Andy Bell has a good one you can look up and use.
  9. Instead of component widths use a single max width in rem on the component.
  10. Font size must never be in px
  11. The button should be full width (100%) and have no height either. Use padding.
  12. Media queries must be defined in rem or em not px so that it works for those who have a different text size too.
  13. I expect your media query is set at too small a value too. There will not be room for the large screen layout at 425px wide.
  14. Don't use absolute positioning for the footer attribution. Let it sit in the dom normally. Never choose arbitrary values to position elements.

  0
• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225Posted 9 months ago

  Hello there 👋

  Good job on completing the challenge !

  Your project looks really good!

  I have a suggestion about your code that might interest you.

  There is an very useful browser extension called Perfect Pixel that allow you compare with the design image and thus see the exact dimensions. I recommend it to you.

  Consider changing the background-color of the body element and the div with theclass of card to hsl(0, 0%, 100%), then add a box-shadow to the section with the class of card_component.

  body {
    background-color: hsl(0, 0%, 100%);
  }
  
  .card_component {
    background-color: hsl(0, 0%, 100%);
    box-shadow: 2px 2px 20px 0 rgba(0, 0, 0, 0.1);
  }
  

  I hope this suggestion is useful for future projects.

  Keep up the excellent work and continue to challenge yourself with new projects. Your progress is impressive, and each project is a step forward in your front-end development journey! 🚀🌟.

  Other than that, great job!

  Happy coding.

  0

  ---

  

  T

  Grace•31,270

  @grace-snowPosted 9 months ago

  @Ezekiel225 I think you should read this post about "pixel perfection" and please stop recommending this extension to everyone: https://www.joshwcomeau.com/css/pixel-perfection/

  Using extensions like that is a waste of time and developers would not do that I a real work flow. And this person you're giving feedback to has pro membership so they already have full access to the design files and all sizes/measurements.

  0