https://codepen.io/CrazyDiamond4811/full/dyoaoNa Please give me feedback.
I would suggest a few things:
- Put a max width on the text content. Too many words per line is hard to read. Also, add some line height so the lines have a little more space between them.
- A lot of people frown upon having large blocks of text centered. I would change it to left justified.
Thanks for the answer.
Iâll fix this.
Well, not as good as it could be. Putting a percentage on the max width still allows for very wide content in browsers that are opened very wide. What other units do you think you could use to limit the max width (and donât say pixels). OK, you can say pixels, most people will. But for a truly responsive design, I would use a different unit.
Also, I think youâll probably want to center the content on the page as well.
I should use âemâ for max-width?
how i center the content on the page?
Yes, I like âemâ better because it allows you to control the number of characters per line better. But like I said, most people would use pixels, so feel free to do that as well.
For your second question, google âCSS centering contentâ. In fact, I would suggest that the first thing you do with any question is to google it and see if you can figure it out on your own. Because thatâs what professionals actually spend quite a bit of time doing, so you might as well get in a lot of practice
Thanks.
Google will be my best friend.
As far as the issues I brought up, yes, they look much better now, donât you think?
Personally, I think you should break that one big block of text into multiple paragraphs because a lot of people are going to find it hard to read as it is. And youâve got some typos (such as the âCarrerâ header).
It does pass all the tests, so I think you are good to go there. But as far as âis it good?â I think thatâs for you to decide.
Good luck and have fun.
Thanks for the help.
Welcome to the forums @Crazycamper. Your page looks good. This is a nit but codepen only expects the code youâd put within the <body> </body>
tags in HTML. (No need to include the body tags). For anything you want to add to <head>
click on the âSettingsâ button, then HTML and add it into the âStuff for <head>â box.
- I mention because you have
<head> </head>
tags in what would essentially be thebody
and they donât belong there. If you want to add a title click the âSettingsâ button and add it there.
Thanks.
Iâll fix this.
I think it looks nice! Nothing I would change