Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verbeteren: Alert #981

Open
8 of 19 tasks
JoeriRoijenga opened this issue Dec 4, 2024 · 1 comment
Open
8 of 19 tasks

Verbeteren: Alert #981

JoeriRoijenga opened this issue Dec 4, 2024 · 1 comment

Comments

@JoeriRoijenga
Copy link
Contributor

Omschrijving

De Alert component kan een aantal verbeteringen ondergaan.

Verbeteringen

documentatie verbeteren op basis van template

  1. WCAG eisen toevoegen.
  2. Aangeven dat het om een Rijkshuisstijl of Utrecht component gaat.

Checks

Definition of ready

  • Naam van de component is duidelijk.
  • Doel van component is beschreven.
  • Design is aanwezig.
  • Tokens zijn aanwezig.
  • Waar in de templates wordt deze component gebruikt? Zijn er praktijkvoorbeelden waarin de component al gebruikt wordt?
  • Beschrijving gemaakt van wat er moet gebeuren om de taak succesvol af te ronden.
  • Testcases / stappen om handmatig te testen.

Definition of done

Component

  • Component is beschikbaar in de index.
  • Design tokens worden gebruikt.

Storybook

  • Tenminste 1 default story.
  • States en properties stories.
  • Documentatie is toegepast, check template.

Tests

  • Het component rendert.
  • Tenminste de Basis functionaliteit van het component wordt getest.

Waar komt de component van:

  • Utrecht
  • Utrecht + wrapper
  • RHC
  • Amsterdam
  • Amsterdam + wrapper
@Rerbun
Copy link
Contributor

Rerbun commented Dec 4, 2024

In demo implementaties was dit component erg goed te gebruiken 👍

Op 1 situatie na, in deze situatie moest alert meerdere headers, de manier waarop ik dat opgelost heb was wel wat hacky

  <Alert type="warning" headingLevel={4} heading={'De periode om in verzet te gaan is voorbij.'}
      textContent={
          <>
              <Paragraph>
                  U bent te laat om in verzet te gaan. Als u toch nog in verzet gaat, maar geen goede reden heeft waarom u te laat bent, wordt uw verzet 'niet-ontvankelijk' verklaard. De officier kijkt dan niet meer naar waarom u vindt dat de straf onterecht is. De straf blijft dan in stand.
              </Paragraph>
              <Heading level={4} appearance="utrecht-heading-5">Wilt u tóch in verzet gaan?</Heading>
              <Paragraph>
                  Leg hieronder uit waarom u niet op tijd in verzet kon gaan.{' '}
                  {/* Als u bewijzen hebt, voeg die dan toe.*/}
                  De officier beoordeelt uw verhaal en bepaalt of uw verzet toch in behandeling wordt genomen of dat uw verzet 'niet-ontvankelijk' wordt verklaard.
              </Paragraph>
          </>
      }
  />

Achteraf had de 2e heading misschien level 5 moeten worden, maar desondanks is het een beetje gek dat dit rendert als een Paragraph met dáárin een paragraph en een heading en nog een paragraph.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants