Fixes #6 - Add media mention banner

static/css/index.css

 #content
 {
   padding-top: 5em;
   margin: 0 auto;
 }
 
 #key-facts h1
 {
   font-size: 16px;
 }
@@ skipping 674 matching lines @@
   background-color: #fff;
   border: 2px solid #a1a1a1;
 }
 
 #ready-for-windows-notice
 {
   margin-bottom: 30px;
 }
 
 /******************************************************************************
  * #media

[juliandoucette, 2018/02/19 14:15:56]

Detail/TOL: This widget also appears on eyeo.com. And I suspect it will occur
again on other websites in the future. Therefore, I think we should implement it
as consistently as we can across websites.

[ire, 2018/02/20 10:49:05]

Noted & Agreed. Is there something in particular you want me to change?

[juliandoucette, 2018/02/21 11:22:53]

Not specifically. Ideally, the solution that you implement here should be
portable to eyeo.com. As a result, you may keep that in mind and test it on
eyeo.com while you are implementing if you choose.

[ire, 2018/02/22 08:43:52]

Ack. The layout between the two sites is noticeably different though. Since the
"As mentioned on" section is to the left. I see your point here though, and I
did actually use the solution from eyeo.com partly here and changed what was
different.

  *****************************************************************************/
 
+#media-section { clear: both; }
+
 #media-header h2,
 #media-list
 {
   margin: 0;
 }
 
 #media-header
 {
   margin-bottom: 2em;
 }
 
 #media-header h2
 {
-  font-size: 100%;

[juliandoucette, 2018/02/19 14:15:57]

Why 100% instead of 1em?

[ire, 2018/02/20 10:49:05]

I have no idea lol. Changed.

+  font-size: 1em;
   font-weight: normal;
   text-transform: uppercase;
 }
 
 #media-list li
 {
   display: inline-block;
 }
 
 #media-list img
 {
-  max-width: 5em;

[juliandoucette, 2018/02/19 14:15:56]

Why max heigh/width instead of *just* hight/width?

[ire, 2018/02/20 10:49:05]

Because not all the images will have the same height/width. But we want them to
fit within a certain "box". If I were to set a specific width, the landscape
images will appear so much smaller than the portrait images and vice versa.

[juliandoucette, 2018/02/21 11:22:54]

Acknowledged.

+  max-width: 10em;
   max-height: 3em;
   margin: 0 1em 1em;
 }
 
 #media-list a 
 {
   position: relative;
   display: block;
 }
 
@@ skipping 31 matching lines @@
   }
 
   #media-list li
   {
     display: table-cell;
     vertical-align: middle;
   }
 
   #media-list img
   {
     margin: 0 2em;  

[juliandoucette, 2018/02/19 14:15:57]

Is this meant to make not display:flex look a little better? Otherwise, it seems
to just add a little space around the list itself?

[ire, 2018/02/20 10:49:04]

It's for both cases, we need some space around each item/image. Or maybe I
misunderstand what your issue is?

[juliandoucette, 2018/02/21 11:22:53]

I was confused because the space applied to each item does not make a difference
when the space provided by display:flex is greater; except for the first space
before the first item and the last space after the last item.

[ire, 2018/02/22 08:43:52]

Acknowledged.

   }
 }
 
 @media (min-width: 992px)
 {
   #media {
     display: table;
     width: 100%;
   }
 
   #media-header,
   #media-list
   {
     display: table-cell;
     vertical-align: middle;
     width: auto;
   }
 
   #media-header
   {
     margin-bottom: 0;
   }
 
   #media-list { direction: rtl; }

[juliandoucette, 2018/02/19 14:15:57]

Why are these necessary?

[ire, 2018/02/20 10:49:05]

To align the list items to the right if flex is not supported. Otherwise the
images will be aligned left. It's the only way I could come up with to not have
the list items aligned left and have a similar look to the flex version.

[juliandoucette, 2018/02/21 11:22:53]

I think that you should be able to make them identical using a non-flex layout.
I may be mistaking.

[ire, 2018/02/22 08:43:52]

I was having trouble achieving the exact same spacing because we need the images
to be vertically centred, and the only non-flex way I could come up with was
table layout. And this forced my hand with the spacing between each image.

It should be noted that none of our "officially" supported devices don't support
flexbox (only the devices we have "partial" support for, i.e. IE 9, IE 8, will
see this version). 

Since this is a NIT, I will opt to keep this solution until I can think of a
better way.

[juliandoucette, 2018/02/23 13:46:17]

Agreed.

   [dir="rtl"] #media-list { direction: ltr; }
 
   @supports (display: flex)

[juliandoucette, 2018/02/19 14:15:57]

There seems to be too little space between items without display:flex on
desktop.

[ire, 2018/02/20 10:49:05]

How much space should there be? The only reason there is so much space between
items with flex enabled is because they can take up all the available space,
which we can't do in the non-flex version. 

[juliandoucette, 2018/02/21 11:22:53]

See answer above. This may or may not be an issue / blocking issue.

   {
     #media-list
     {
       display: flex;
       justify-content: space-between;
       align-items: center;
     }
   }
 
 }