Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found

Submitted

html and css

@umahichristopher

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi friends I am a beginner in HTML and CSS, Please tell me how I can improve in HTML and CSS

Community feedback

Hassia Issah 50,670

@Hassiai

Posted

Replace<div class="container">with the main tag, <h3> with <h1> and <div class="attribution"> with the footer tag to fix the accessibility issues. click here for more on web-accessibility and semantic html

To make the image visible remove "images/". <img id="ivaad" src=" image-qr-code.png" alt="iamge">

The body has a wrong background-color. Use the colors that were given in the styleguide.md found in the zip folder you downloaded.

To center .container on the page using flexbox or grid instead of a margin, add min-height:100vh; display: flex; align-items: center: justify-content: center; or min-height:100vh; display: grid place-items: center to the body.

To center .container on the page using flexbox:
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
To center .container on the page using grid:
body{
min-height: 100vh;
display: grid;
place-items: center;
}

For a responsive content, give .container a fixed max-with value and a padding value for all the sides and a border-radius. Give the img a max-width of 100% for a responsive image.

Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here

Reduce the border-radius values.

Give h1 and p the same font-size of 15px, text-align: center,the same margin-left, margin-right and margin-top values. Give p a margin bottom value.

Hope am helpful.

Well done for completing this challenge. HAPPY CODING

Marked as helpful

0

@umahichristopher

Posted

@Hassiai thanks

0
Elona 60

@devel0na

Posted

You could use the font suggested on the style-guide.md by importing it from Google Fonts!

  • Go to the font page from Google Fonts.
  • In the 'Styles' section, there are different types of weights. Select the ones that the style-guide.md suggests. When selecting a weight, a drawer will show on the right side of the page.
  • Go to the 'Use on the web' section in the drawer. Then, copy the @import inside of the style tag. Copy this code on the top of your styles.css file.
  • Now you have imported the font. To include it in the project (and for practical usage) this font could be applied to the body tag.
  • To set the font, go to the 'CSS rules to specify families' section in the drawer, copy the rule, and then paste it into the selector that you use for styling the body tag.

Hope this works for you! :)

Marked as helpful

0

@umahichristopher

Posted

@devel0na Thanks

0

Please log in to post a comment

Log in with GitHub
Discord logo

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