# Assessment wanted for Fundamental Layout Comprehension-jtilly

**URL:** <https://discourse.mozilla.org/t/assessment-wanted-for-fundamental-layout-comprehension-jtilly/91851>\
**Category:** Learn\
**Created:** [January 22, 2022, 3:45am UTC](https://discourse.mozilla.org/t/assessment-wanted-for-fundamental-layout-comprehension-jtilly/91851 "2022-01-22T03:45:17Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![jtilly](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/jtilly/32/51946_2.png) [@jtilly](https://discourse.mozilla.org/u/jtilly)\
**Post date:** [January 22, 2022, 3:45am UTC](https://discourse.mozilla.org/t/assessment-wanted-for-fundamental-layout-comprehension-jtilly/91851/1 "2022-01-22T03:45:17Z")

</div>

Hi all!

I’d like feedback on my solution to [Test Your Skills: Fundamental layout comprehension](https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Fundamental_Layout_Comprehension)

  

**Here is my code:**

> **[GitHub - jasheloper/css-layout: MDN skill assessment - Fundamental layout...](https://github.com/jasheloper/css-layout)**
>
> MDN skill assessment - Fundamental layout comprehension

  

Thank you! 🙂

---

<div class="post-metadata">

**Author:** ![mikoMK](https://sea1.discourse-cdn.com/flex001/user_avatar/discourse.mozilla.org/mikomk/32/50456_2.png) [@mikoMK](https://discourse.mozilla.org/u/mikoMK)\
**Post date:** [January 24, 2022, 7:39am UTC](https://discourse.mozilla.org/t/assessment-wanted-for-fundamental-layout-comprehension-jtilly/91851/2 "2022-01-24T07:39:27Z")

</div>

Hi @jtilly

Amazing job on this task! 🏅  
Compliments on the additional work to make it look good on smaller screens.

Something I would change:  
Flexbox on `<main>` makes the `<aside>` shrink to its minimum width. Maybe that’s fine for this layout. To get more control over the width you could instead use grid on `<main>`. Something like this would work:

```auto
display: grid;
grid-template-columns: 3fr 1fr;
grid-gap: 20px;

```

This would also require to change to a one column layout on smaller screens.

I hope that helps! 🙂

Have a nice day,  
Michael