@cpaz Great start! I like the simplicity of the page. Your colors pass the contrast checker and your HTML and CSS also pass the validators, so that’s good. It’s also good that your image has an
You probably left this in accidentally, but this
<li class = ""> can be reduced to just
<li>. Or, if you wanted to stay true to separate the content from the design, you could leave the class attribute, remove the <
strong> tags, and create a class in your CSS called
listItemDate or something like that and make the font weight bold.
To answer your question, yes, it’s definitely good practice to size heading elements with CSS:
Heading information may be used by user agents, for example, to construct a table of contents for a document automatically.
Do not use lower levels to decrease heading font size: use the CSS
font-size property instead.
Avoid skipping heading levels: always start from
<h1>, next use
<h2> and so on.
You should consider avoiding using
<h1> more than once on a page. See “Defining sections” in Using HTML sections and outlines for more information.
To fix your background color problem (assuming you want the background color to cover the page completely – I currently see white background on the sides), you’ll want to structure your HTML with basic and necessary HTML elements. Every HTML page needs to have
<body> elements (1 | 2). Once you’ve added these elements, you can select the
body elements in your CSS and give them the black background color.
Good work so far!