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.

Sign up for GitHub

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

Jump to bottom

Verbeteren: Action Group #980

Open
13 of 19 tasks
JoeriRoijenga opened this issue Dec 4, 2024 · 2 comments
Open
13 of 19 tasks

Verbeteren: Action Group #980

JoeriRoijenga opened this issue Dec 4, 2024 · 2 comments

Comments

@JoeriRoijenga
Copy link
Contributor

JoeriRoijenga commented Dec 4, 2024 •
edited by Aref-Akminasi
Loading

Omschrijving

De Action Group component kan een aantal verbeteringen ondergaan.

Verbeteringen

documentatie verbeteren op basis van template

  1. Teksten grondig bekijken, deze lijken elkaar tegen te spreken.
  2. Kijken of terminologie als koptekst nodig is en verwijderen van verticale variant(?) tekst.
  3. WCAG eisen toevoegen (afhankelijk van Utrecht docs/NLDS
  4. Documentatie updaten (Gebruik voor / niet gebruiken voor / anatomie) template
  5. RHC space tokens voor row gap en column gap toevoegen in SCSS
  6. Link naar NL Design System pagina.
  7. Aangeven dat het om een Rijkshuisstijl of Utrecht component gaat.
  8. Doel toevoegen van component.
  9. Geen voorbeelden van goede en foute implementaties., komt mogelijk later.
  10. Code snippet toevoegen
  11. Rename "default" naar "default (row)"# 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
@JoeriRoijenga JoeriRoijenga converted this from a draft issue Dec 4, 2024
@Rozerinay Rozerinay moved this from Todo to Ready in Community Sprint - Rijkshuisstijl Dec 13, 2024
@Rerbun Rerbun self-assigned this Jan 9, 2025
@Rerbun Rerbun moved this from Ready to In Progress in Community Sprint - Rijkshuisstijl Jan 9, 2025
@Rerbun Rerbun mentioned this issue Jan 15, 2025
Merged
@Rerbun Rerbun moved this from In Progress to On hold in Community Sprint - Rijkshuisstijl Jan 15, 2025
@Rerbun Rerbun removed their assignment Jan 15, 2025
@Rerbun
Copy link
Contributor

Rerbun commented Jan 15, 2025

Ik verplaats deze naar On Hold, bij #1010 is naar voren gekomen dat de documentatie van Utrecht nog niet genoeg gevuld is om te kunnen importeren

Rerbun added a commit that referenced this issue Jan 17, 2025
@Rerbun
feat(storybook): improved accordion and action-group docs and unit te…
801910d 
…sts (#1117)

#980 en #979

---------

Co-authored-by: Ruben Smit <[email protected]>
@Aref-Akminasi Aref-Akminasi moved this from On hold to Todo in Community Sprint - Rijkshuisstijl Jan 21, 2025
WebBeest pushed a commit that referenced this issue Jan 27, 2025
@Rerbun
feat(storybook): improved accordion and action-group docs and unit te…
8cc1647 
…sts (#1117)

#980 en #979

---------

Co-authored-by: Ruben Smit <[email protected]>
WebBeest pushed a commit that referenced this issue Jan 27, 2025
@Rerbun
feat(storybook): improved accordion and action-group docs and unit te…
52997c0 
…sts (#1117)

#980 en #979

---------

Co-authored-by: Ruben Smit <[email protected]>
@Aref-Akminasi Aref-Akminasi moved this from Ready to In Progress in Community Sprint - Rijkshuisstijl Feb 3, 2025
@Aref-Akminasi Aref-Akminasi self-assigned this Feb 3, 2025
@Aref-Akminasi Aref-Akminasi moved this from In Progress to On hold in Community Sprint - Rijkshuisstijl Feb 4, 2025
@Aref-Akminasi Aref-Akminasi removed their assignment Feb 4, 2025
@Aref-Akminasi Aref-Akminasi moved this from On hold to Todo in Community Sprint - Rijkshuisstijl Feb 6, 2025
@Rerbun
Copy link
Contributor

Rerbun commented Feb 11, 2025

Custom tokens nodig ipv common tokens: #1196 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Community Sprint - Rijkshuisstijl
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Rerbun @JoeriRoijenga @Aref-Akminasi