seb-oss/green

Grid: Gap issues

Closed this issue · 2 comments

Hi Astrit,

I found some issues in you fabulous grid.

  1. There are no gap space in the horizontal (row) line. (see picture)
  2. Specifying different values for the gap (Example: gap="l:xl m:l s:xs") doesn't have any effect. Alvays 32px for all view ports.

Bonus feedback. https://sebgroup.github.io/green/latest/core/?path=/docs/components-grid--page
The values on columns should have the size/media value besides the amount of columns.
image

<gds-grid columns="8 4 2" auto-columns="l:200 m:100 s:80">


(columns="l:8 m:4 s:2").

image

@astrit

Hi @daggelito02

Super nice that you already started to use Grid, and happy to know you like it.

Now to answer your questions:

  1. There isn't at the moment a functionality to add row gap, we might introduce that feature in the future this could be a good test, how ever it is meant to do that by adding margin-block-end to the child it self at least for now.
  2. That is a bit odd on my setup it works and I did test it out again to make sure but we can take a closer look at your setup and try to debug.

Here is an example on the core-lib-dev
CleanShot 2024-04-08 at 23 23 52@2x

Good catch on the last one as that is a typo actually only on the documentation as in fact still needs the device prefix.

Let me know when we can check on this!

🤘

@daggelito02 these issues were addressed on PR: #1302 , which has been published. Please upgrade core to 1.12.0

The release:
https://github.com/sebgroup/green/releases/tag/%40sebgroup%2Fgreen-core%401.12.0