Hello,
Here’s my code: Edit fiddle - JSFiddle - Code Playground
I have had a go at this assessment and I think I have done a pretty good job.
I would appreciate any feedback and suggestions on how I can improve.
Thanks ![]()
Hello,
Here’s my code: Edit fiddle - JSFiddle - Code Playground
I have had a go at this assessment and I think I have done a pretty good job.
I would appreciate any feedback and suggestions on how I can improve.
Thanks ![]()
Hi @Ahmed1
This looks great! Congratulations 
As a more modern approach you could also use a grid on the main element instead of the floats for <article> and <aside>:
.grid {
display: grid;
grid-template-columns: 3fr 1fr;
grid-gap: 20px;
}
flex-direction: row; can be omitted since row is the default value.
Feel free to ask if anything is unclear 
Michael
Thanks!!
Changed it so now the webpage is now using the modern grid approach instead of floats, appreciate your help 