Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: static/scss/components/_groups.scss

Issue 29453600: Issue 4961 - Fix card group field alignment on acceptableads.com (Closed) Base URL: https://hg.adblockplus.org/web.acceptableads.com
Left Patch Set: Rebase Created July 17, 2017, 1:41 p.m.
Right Patch Set: Move styling for cards on committee page to committee/index.html Created July 25, 2017, 1:33 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « static/scss/_variables.scss ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 // This file is part of acceptableads.org. 1 // This file is part of acceptableads.org.
2 // Copyright (C) 2016 Eyeo GmbH 2 // Copyright (C) 2016 Eyeo GmbH
3 // 3 //
4 // acceptableads.org is free software: you can redistribute it and/or modify 4 // acceptableads.org is free software: you can redistribute it and/or modify
5 // it under the terms of the GNU General Public License as published by 5 // it under the terms of the GNU General Public License as published by
6 // the Free Software Foundation, either version 3 of the License, or 6 // the Free Software Foundation, either version 3 of the License, or
7 // (at your option) any later version. 7 // (at your option) any later version.
8 // 8 //
9 // acceptableads.org is distributed in the hope that it will be useful, 9 // acceptableads.org is distributed in the hope that it will be useful,
10 // but WITHOUT ANY WARRANTY; without even the implied warranty of 10 // but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 282 matching lines...) Expand 10 before | Expand all | Expand 10 after
293 } 293 }
294 // un-center all cards 294 // un-center all cards
295 .group > div:nth-child(4), 295 .group > div:nth-child(4),
296 .group > div:nth-child(3), 296 .group > div:nth-child(3),
297 .group > div:nth-child(5) 297 .group > div:nth-child(5)
298 { 298 {
299 margin-left: 0; 299 margin-left: 0;
300 } 300 }
301 } 301 }
302 302
303 @supports (display: grid) 303 @supports (display: grid)
juliandoucette 2017/07/18 22:07:36 Something funky seems to be going on at 992px wide
ire 2017/07/20 17:43:25 Done.
304 { 304 {
305 305
306 .group.cards 306 .group.cards
307 { 307 {
308 display: grid; 308 display: grid;
309 grid-template-columns: repeat(1, 1fr); 309 grid-template-columns: repeat(1, 1fr);
310 310
311 @media (min-width: $card-min-width) 311 @media (min-width: $card-min-width)
312 { 312 {
313 // Each card should be a minimum width of the $card-min-width. Otherwise, take up available space 313 // Each card should be a minimum width of the $card-min-width. Otherwise, take up available space
314 grid-template-columns: repeat(auto-fit, minmax($card-min-width, 1fr)); 314 grid-template-columns: repeat(auto-fit, minmax($card-min-width, 1fr));
315 } 315 }
316 } 316 }
317 317
318 .group.cards .card 318 .group.cards .card
319 { 319 {
320 display: inline-grid; 320 display: inline-grid;
321 width: 100%; 321 width: 100%;
322 grid-template-rows: auto 1fr auto; 322 grid-template-rows: auto 1fr auto;
323 } 323 }
324 324
325 .group.cards .card-summary 325 .group.cards .card-summary
326 { 326 {
327 @media all 327 height: auto;
juliandoucette 2017/07/18 22:07:36 NIT: `@media all` doesn't seem to make a differenc
ire 2017/07/20 17:43:25 Yep you're right :p . Done.
328 {
329 height: auto;
330 }
331 } 328 }
332 329
333 .group-4.cards 330 .group-4.cards
334 { 331 {
335 @media (min-width: $desktop-breakpoint) 332 @media (min-width: $desktop-breakpoint)
336 { 333 {
337 // Each card should take a minimum of 25% width of the group 334 // Each card should take a minimum of 25% width of the group
338 grid-template-columns: repeat(auto-fit, minmax(25%, 1fr)); 335 grid-template-columns: repeat(auto-fit, minmax(25%, 1fr));
339 } 336 }
340 } 337 }
341 338
342 .group-5.cards 339 .group-5.cards
343 { 340 {
344 @media (min-width: $tablet-breakpoint) 341 @media (min-width: $tablet-breakpoint)
345 { 342 {
346 grid-template-columns: repeat(auto-fit, minmax(33.3%, 1fr)); 343 grid-template-columns: repeat(auto-fit, minmax(33.3%, 1fr));
347 } 344 }
348 @media (min-width: $desktop-breakpoint) 345 @media (min-width: $desktop-breakpoint)
349 { 346 {
350 grid-template-columns: repeat(auto-fit, minmax(20%, 1fr)); 347 grid-template-columns: repeat(auto-fit, minmax(20%, 1fr));
351 } 348 }
352 } 349 }
353 350
354 .group-5.cards > div:nth-child(4), 351 .group-5.cards > div:nth-child(4),
355 .group-5.cards > div:nth-child(5) 352 .group-5.cards > div:nth-child(5)
356 { 353 {
357 @media all 354 margin-left: 0;
358 {
359 margin-left: 0;
360 }
361 } 355 }
362 356
363 .group-5.cards > div:nth-child(4) 357 .group-5.cards > div:nth-child(4)
364 { 358 {
365 @media (min-width: $tablet-breakpoint) and (max-width: $desktop-breakpoint) 359 @media (min-width: $tablet-breakpoint) and (max-width: $desktop-breakpoint - 1px)
366 { 360 {
367 margin-left: 50%; 361 margin-left: 50%;
368 } 362 }
369 } 363 }
370 364
371 .group-5.cards > div:nth-child(5) 365 .group-5.cards > div:nth-child(5)
372 { 366 {
373 @media (min-width: 700px) and (max-width: $desktop-breakpoint) 367 @media (min-width: 700px) and (max-width: $desktop-breakpoint - 1px)
374 { 368 {
375 margin-left: 50%; 369 margin-left: 50%;
376 } 370 }
377 } 371 }
378 372 }
379 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld