Refactor CSS: Add styles and remove comments

Removed starter comments and added styles for page header and article layout.

Author: Motorcycle-lab

Wireframe/style.css

@@ -1,21 +1,17 @@
-/* Here are some starter styles
-You can edit these or replace them entirely 
-It's showing you a common way to organise CSS 
-And includes solutions to common problems 
-As well as useful links to learn more */
+.page-header{
+display:flex;
+flex-direction:column;
+align-items: center;
+justify-content: center;
+text-align: center;
+min-height: 300px;
+background-color: #f4f4f4;  
+}
+
+.page-header h1 {
+  margin-bottom: 10px;
+}
 
-/* ====== Design Palette ======
- This is our "design palette".
- It sets out the colours, fonts, styles etc to be used in this design 
- At work, a designer will give these to you based on the corporate brand, but while you are learning
- You can design it yourself if you like
- Inspect the starter design with Devtools
- Click on the colour swatches to see what is happening
- I've put some useful CSS you won't have learned yet
- For you to explore and play with if you are interested
- https://web.dev/articles/min-max-clamp
- https://scrimba.com/learn-css-variables-c026
-====== Design Palette ====== */
 :root {
   --paper: oklch(7 0 0);
   --ink: color-mix(in oklab, var(--color) 5%, black);
@@ -24,8 +20,7 @@ As well as useful links to learn more */
   --line: 1px solid;
   --container: 1280px;
 }
-/* ====== Base Elements ======
- General rules for basic HTML elements in any context */
+
 body {
   background: var(--paper);
   color: var(--ink);
@@ -41,11 +36,8 @@ svg {
   width: 100%;
   object-fit: cover;
 }
-/* ====== Site Layout ====== 
-Setting the overall rules for page regions
-https://www.w3.org/WAI/tutorials/page-structure/regions/
-*/
-main {
+
+.main-wrapper {
   max-width: var(--container);
   margin: 0 auto calc(var(--space) * 4) auto;
 }
@@ -54,36 +46,51 @@ footer {
   bottom: 0;
   text-align: center;
 }
-/* ====== Articles Grid Layout ====
-Setting the rules for how articles are placed in the main element.
-Inspect this in Devtools and click the "grid" button in the Elements view
-Play with the options that come up.
-https://developer.chrome.com/docs/devtools/css/grid
-https://gridbyexample.com/learn/
-*/
-main {
+
+.main-wrapper {
   display: grid;
   grid-template-columns: 1fr 1fr;
   gap: var(--space);
   > *:first-child {
     grid-column: span 2;
   }
 }
-/* ====== Article Layout ====== 
-Setting the rules for how elements are placed in the article. 
-Now laying out just the INSIDE of the repeated card/article design.
-Keeping things orderly and separate is the key to good, simple CSS.
-*/
+
 article {
   border: var(--line);
   padding-bottom: var(--space);
-  text-align: left;
-  display: grid;
-  grid-template-columns: var(--space) 1fr var(--space);
-  > * {
-    grid-column: 2/3;
-  }
-  > img {
-    grid-column: span 3;
-  }
+ display: flex;
+ flex-direction: column;
+  
+  height: 100%;
+  
+  padding: 0 var(--space) var(--space) var(--space);
+  
+  overflow:hidden;
+}
+
+article img{
+  width: calc(100% + (var(--space) *2));
+  margin-left: calc(var(--space)* -1);
+  height: 250px;
+  object-fit: cover;
+  margin-bottom: var(--space);
+}
+
+article h2 {
+  min-height: 5em; 
+  display: flex;
+  align-items: center; /* Vertically centers the text if it's only one line */
+  margin-top: 0;
+  margin-bottom: var(--space);
+}
+
+article p{
+  flex-grow: 1;
+  margin-bottom: var(--space);
+}
+
+article a{
+  margin-top: auto;
+  align-self: flex-start;
 }