Issue 4961 - Fix card group field alignment on acceptableads.com

static/scss/components/_groups.scss

 // This file is part of acceptableads.org.
 // Copyright (C) 2016 Eyeo GmbH
 //
 // acceptableads.org is free software: you can redistribute it and/or modify
 // it under the terms of the GNU General Public License as published by
 // the Free Software Foundation, either version 3 of the License, or
 // (at your option) any later version.
 //
 // acceptableads.org is distributed in the hope that it will be useful,
 // but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 263 matching lines @@
     width: 100%;
   }
   // un-center all cards
   .group > div:nth-child(4),
   .group > div:nth-child(3),
   .group > div:nth-child(5)
   {
     margin-left: 0;
   }
 }
+
+@supports (display: grid) 
+{
+
+  .group
+  {
+    display: grid;
+    // Each card should be a minimum width of the $card-max-width. Otherwise, take up available space
+    grid-template-columns: repeat(auto-fit, minmax($card-max-width, 1fr));
+  }
+
+  .group .card 
+  {
+    display: inline-grid;
+  }
+
+  .group-4 
+  {
+    @media (min-width: $desktop-breakpoint) 
+    {
+      // Each card should take a minimum of 25% width of the group
+      grid-template-columns: repeat(auto-fit, minmax(25%, 1fr));
+    }
+  }
+
+  .group-5 

[juliandoucette, 2017/06/16 19:39:17]

I'm not sure how/why yet - but this patchset seems to break the centring of the
last card in 5 & 3 card groups. 

[ire, 2017/06/21 07:47:58]

I've fixed the centering of the last card in 5 card groups, but I don't see any
uses of 3 card groups (unless you count this page -
http://localhost:5000/en/committee - which is actually a `.group-4`). Plus,
there doesn't seem to be a centering of the last card in 3 card groups looking
at the CSS.

+  {
+    @media (min-width: $desktop-breakpoint) 
+    {
+      grid-template-columns: repeat(auto-fit, minmax(20%, 1fr));
+    }
+  }
+
+  .group-5 > div:nth-child(4) 

[ire, 2017/06/13 16:51:23]

Since this is overwriting a previous style, I stuck with the `> div` selector

[juliandoucette, 2017/06/16 19:39:17]

Acknowledged.

+  {
+    @media all 

[juliandoucette, 2017/06/16 19:39:17]

Why @media all?

[ire, 2017/06/21 07:47:58]

See comment above

+    {
+      margin-left: 0;
+    }
+  }
+
+}