A responsive web HTML5 and CSS design using grid.

Community feedback

• Account deletedPosted over 1 year ago

  With the span its a inline element so margin and padding wont effect it. Either set the display to block on the span, or simply change it to a p tag. Only use span tags if you are adding something directly into a h1-h6 tag or a p tag. Otherwise you dont really ndeed to use them. Hope this helps.

  11

  Marked as helpful
• 

  Mohammed Fakih•1,590

  @javascriptor1

  Posted over 1 year ago

  Hi Peace,

  Nice job. Here are some notes about your solution :

  • HTML

  1- For HTML , always use semantic tags and avoid using generic tags like div and span as much as possible unless you have no other options. Example , instead of div tag you have used for container , use main tag and use H1 instead of P tag for the first senetence. I have opened a PR for this purpose. Please check your github repo for this project for more info.

  2- Add alternative text alt to the image so you make it accessible. People with disabilities like blindess can't see images. They rely instead on screen reader to consume and interact with the content. Alt is the texts that represent the image in your page. Write a text which can express what is this photo and convey the meaning for it. For this challenge , you can use something like alt="QR code for frontend mentor website"

  • CSS

  3- Media query you wrote is redudant and has no effect as its the same styling which is already applied for the container on different screen sizes. Remove it or change the style if you want to have some changes for mobile screens.

  4- Another repeated styling which can be removed is the

  margin-top: 4rem; for container. You have already added margin: 4rem;

  5- for P tag , font-weight: 400; also can be removed as this is the default value. Same for display: block. p tag is a block level element by default.

  Best of luck

  02

  Marked as helpful
• 

  bright•120

  @bright-op

  Posted over 1 year ago

  <span> is by default an inline element, meaning, you can only space a span element left and right, use: padding or margin to add spacing to left and right. To add spacing top and down, you need to use display:block

  01

  Marked as helpful