Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
82 changes: 52 additions & 30 deletions .github/ISSUE_TEMPLATE/202-📝-varianten-bepaald.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,11 @@ Component is, waar nodig, versimpeld of opgesplitst. Zodat er één of meerdere

Bijvoorbeeld voor Paragraph:

| Paragraph | Amsterdam | Den Haag | DUO | Open Formulieren | Open Gemeenten | Rotterdam | RVO | Utrecht |
| Lead/Big/Header paragraph/Intro-text | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ |
| Small/Details Small/Small print | ✅ | ✅ | - ✅ | - | - | ✅ | ✅ |
| XL | - | - | - | - | - | - | ✅ | - |
| Paragraph | Amsterdam | Den Haag | DUO | Open Formulieren | Open Gemeenten | Rotterdam | RVO | Utrecht |
| :----------------------------------- | :-------- | :------- | :-- | :--------------- | :------------- | :-------- | :-- | :------ |
| Lead/Big/Header paragraph/Intro-text | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ | ✅ |
| Small/Details Small/Small print | ✅ | ✅ | ✅ | - | - | - | ✅ | ✅ |
| XL | - | - | - | - | - | - | ✅ | - |

Keuzes:

Expand Down Expand Up @@ -116,12 +117,12 @@ Voorbeeld hypothese voor RVO met 4 verschillende formaten voor font-size:
- Overleg de naam en beschrijving met het kernteam.
- Bepaal samen met de Design System Lead welke varianten in de Candidate component moeten worden opgenomen.
- Is het een variant die hoort bij een standaard HTML attribuut? Bepaal dan welke varianten wel worden ondersteund en ook geadviseerd worden, welke ondersteund worden maar niet geadviseerd worden en welke er niet ondersteund worden.
- Notuleer de besproken beredeneringen. Deze heb je nodig in de volgende stap. Dat doe je met een comment in dit GitHub issue met een comment:
- Notuleer de besproken beredeneringen. Deze heb je nodig in de volgende stap. Dat doe je met een comment in dit GitHub issue met een comment. Hieronder zie je een voorbeeld van Text Input

```md
## Input type
## Type

### ondersteund:
### Ondersteund:

- text
- search
Expand All @@ -130,7 +131,7 @@ Voorbeeld hypothese voor RVO met 4 verschillende formaten voor font-size:
- email
- password

### ondersteund maar niet geadviseerd:
### Ondersteund maar niet geadviseerd:

- hidden
- :not([type])
Expand All @@ -141,7 +142,7 @@ Voorbeeld hypothese voor RVO met 4 verschillende formaten voor font-size:
- datetime-local
- number

### niet ondersteund:
### Niet ondersteund:

- color
- range
Expand All @@ -162,23 +163,35 @@ Voorbeeld hypothese voor RVO met 4 verschillende formaten voor font-size:
```md
## Candidate voorbereidingsfase: Versimpeld of gesplitst

De varianten voor de Candidate component zijn bepaald, dat hebben we gedaan op basis van alle voorbeelden die we in deze Discussion hebben gezien. Als een variant veel voorkomt, dan is het een algemeen nuttige variant!
De varianten voor de Candidate component zijn bepaald, dat is gedaan op basis van alle voorbeelden die we in deze Discussion hebben gezien. Als een variant veel voorkomt, dan is het een algemeen nuttige variant.

### Deze varianten ondersteunen we:
### Deze varianten worden ondersteund

**{variant-groep}** - (enum / boolean / string)
{

#### {variant-groep} - (enum / boolean / string / number)

Is het een enum? Noteer dan de waarden en of er een default is.

- {variant-naam}(default)
- {variant-naam}
- {variant-naam}

### Deze varianten ondersteunen we niet:
Is het een boolean? Noteer dan de twee opties en welke de default is.

- true (default)
- false

Is het een string of number? Laat deze dan leeg.
}

### Deze varianten worden niet ondersteund

optioneel: welke varianten hebben we besloten om niet te supporten? Zie hiervoor de notulen in dit GitHub issue.
{Optioneel: welke varianten hebben we besloten om niet te supporten? Zie hiervoor de notulen in dit GitHub issue.}

### Deze varianten ondersteunen we, maar raden we niet aan:
### Deze varianten worden ondersteund, maar raden we niet aan

optioneel: welke varianten supporten we omdat ze standaard in HTML zitten, maar gaan we niet documenteren, niet beschikbaar maken in Figma en alleen meenemen in de test Storybook? Zie hiervoor de notulen in dit GitHub issue
{Optioneel: welke varianten ondersteunen we omdat ze standaard in HTML zitten, maar gaan we niet documenteren, niet beschikbaar maken in Figma en alleen meenemen in de test Storybook? Zie hiervoor de notulen in dit GitHub issue.}

### Mis je een variant?

Expand All @@ -201,47 +214,56 @@ Er zijn meerdere implementaties van {component-naam} bekeken en met elkaar verge
Maak acceptatiecriteria voor alle varianten die in de component worden meegenomen. Voeg deze toe aan de `acceptance-criteria.md` van de component in de Candidate repository.

```md
## Welke varianten ondersteunen we voor deze component
## Varianten

Deze varianten worden ondersteund. Wanneer ze standaard in HTML zitten worden ze in Typescript gedefinieerd door de attributen het standaard HTMLElement te gebruiken. Wanneer ze niet standaard in HTML zitten wordt deze type extended. Deze varianten worden gedocumenteerd op de website, in de README en in de standaard Storybook. Deze varianten worden beschikbaar gemaakt in Figma.
### Deze varianten worden ondersteund

**{variant-groep}** - (enum / boolean / string / number)
- Varianten die standaard in HTML beschikbaar zijn, worden in Typescript gedefinieerd door de attributen van het standaard HTMLElement te gebruiken.
- Varianten die niet standaard in HTML beschikbaar zijn, worden als uitgebreiding toegevoegd.
- Deze varianten worden gedocumenteerd op de website, in de README en in de standaard Storybook.
- Deze varianten worden beschikbaar gemaakt in Figma.

{
Is het een enum? Noteer dan de waardes en of er een default is

#### {variant-groep} - (enum / boolean / string / number)

Is het een enum? Noteer dan de waarden en of er een default is.

- {variant-naam}(default)
- {variant-naam}
- {variant-naam}

Is het een boolean? Noteer dan de twee opties en welke de default is
Is het een boolean? Noteer dan de twee opties en welke de default is.

- true (default)
- false

Is het een string of number? Laat deze dan leeg
Is het een string of number? Laat deze dan leeg.
}

### Deze varianten ondersteunen we niet:
### Deze varianten worden niet ondersteund

{
optioneel:

Deze varianten die standaard in HTML zitten worden niet ondersteund. Ze worden uit de HTMLElement type gehaald met Omit bij het definieren van de properties van de component in Typescript.

Deze varianten worden ook niet gedocumenteerd en niet meegenomen in Figma.
- Deze varianten, die standaard in HTML beschikbaar zijn, worden niet ondersteund. Ze worden uit het HTMLElement-type gehaald met Omit tijdens het definiëren van de properties in Typescript.
- Deze varianten worden niet gedocumenteerd
- Deze varianten worden niet beschikbaar gemaakt in Figma.

Documenteer de niet ondersteunde varianten op basis van de notulen in dit GitHub issue.
Documenteer de niet-ondersteunde varianten op basis van de notulen in dit GitHub-issue.
}

### Deze varianten ondersteunen we, maar raden we niet aan:
### Deze varianten worden ondersteund, maar raden we niet aan

{
optioneel:

Deze varianten supporten we omdat ze standaard in HTML zitten door de HTMLElement type te extenden in Typescript. Deze varianten worden meegenomen in de test Storybook. Deze varianten worden niet gedocumenteerd op de website of in de README, niet beschikbaar in Figma en niet meegenomen in de standaard Storybook.
- Omdat deze varianten standaard in HTML beschikbaar zijn, worden ze ondersteund. Dit doen we door het HTMLElement-type te extenden in Typescript.
- Deze varianten worden niet gedocumenteerd op de website, in de README en in de standaard Storybook.
- Deze varianten worden wel meegenomen in de 'Test' Storybook.
- Deze varianten worden beschikbaar gemaakt in Figma.

Documenteer de ondersteunde, maar niet aangeraden varianten op basis van de notulen in dit GitHub issue.
Documenteer de ondersteunde, maar niet aangeraden varianten op basis van de notulen in dit GitHub-issue.
}
```

Expand Down
Loading