fix: use slot content for expandable rows in FInteractiveTable (refs SFKUI-6935)
#202
Conversation
|
…SFKUI-6935) fixes #157
MCFK
force-pushed
the
fix/SFKUI-6935-expandable-row
branch
from
5b3d1df to
479942f
Compare
| &--indented { | ||
| padding-left: size.$padding-150; | ||
| } | ||
|
|
There was a problem hiding this comment.
Vågar man ta bort denna?
Ersätts av td:nth-child(2)
There was a problem hiding this comment.
Varför just :nth-child(2)? Påverkas det inte av om man har valbara rader eller ej?
There was a problem hiding this comment.
Jo, insåg just det. Så behöver hitta något bättre att gå på.
MCFK
requested review from
ext,
dvitamin,
NicklasWei,
kristofernilsson,
Marie-Jakobsson,
kristinalindstrom,
mitra-Re,
OlofFredriksson and
TMFKse
as code owners
| &--indented { | ||
| padding-left: size.$padding-150; | ||
| } | ||
|
|
There was a problem hiding this comment.
Varför just :nth-child(2)? Påverkas det inte av om man har valbara rader eller ej?
| <f-table-column name="id" title="Id" shrink> {{ row.id }} </f-table-column> | ||
| <f-table-column name="date" title="Datum" type="date"> {{ row.date }} </f-table-column> | ||
| <f-table-column name="amount" title="Summa" type="numeric"> | ||
| {{ row.sum }} kronor |
There was a problem hiding this comment.
Hade man kunnat skriva ett jest-test eller komponent-test som säkerställer att i det här fallet skrivs det ut exempelvis "200 kr" istället för bara "200" som det blir före buggrättningen?
| @@ -55,31 +55,8 @@ Med expanderbara rader går det att skapa ytterligare tabellrader som visas när | |||
| För att skapa expanderbart innehåll som följer existerande kolumner så krävs det att `expandable-attribute` är satt och `expandable` slot inte används. | |||
| Innehållet måste då följa samma datastruktur som ordinarie rader. | |||
|
|
|||
| Se nedan exempel av data som kan användas för att generera en expanderbar rad som innehåller två tabellrader. | |||
|
|
|||
| ```js static | |||
There was a problem hiding this comment.
Egentligen ser jag gärna att detta finns kvar utöver körbart exempel men det kanske bara är jag? Det jag tycker är nice med att visa statisk kod är att här är det bara den relevanta delen, jag behöver inte mentalt filtrera bort allt irrelevant i exemplet.
https://forsakringskassan.github.io/designsystem/pr-preview/pr-202/components/table-and-list/table.html#expanderbara_rader