Skip to content

Conversation

@jstuyts
Copy link
Contributor

@jstuyts jstuyts commented Dec 15, 2025

Closes #449.

@vercel
Copy link

vercel bot commented Dec 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
rotterdam Ready Ready Preview, Comment Dec 19, 2025 11:25am

@keesvandieren
Copy link
Contributor

Review:

  • Staat het component in de goede module?
    • Is het component generiek of specifiek voor een organisatie? Organisatiespecifieke componenten mogen niet in generieke modules.
    • Is het een lichtgewicht, in veel apps gebruikt component of, is het een zwaar en/of weinig gebruikt component? Zware en/of weinig gebruikte componenten kunnen beter in een 'extras'-module.
  • Staat het in de goede package?
  • Klopt de status in @NlComponentState?
    • Is het component al verder in het NLDS-proces? Dan moet er overgestapt worden van een community- naar een kandidaatcomponent, of van een kandidaatcomponent naar een hall-of-fame-component.
  • Front-end resources:
    • Worden in de POM de minimale front-endbestanden gekopieerd?
    • Is het nodig om het package te openen in module-info.java?
  • Worden de juiste Wicket-typen (Panel, Border, Behavior, enz.) gebruikt voor de implementatie? Er kunnen meerdere typen zijn omdat er verschillende situaties zijn: nieuwe code of aanpassen van bestaande code, gebruik in een basispagina of in een panel, enz.
    • Zijn alle Wicket-constructors overgenomen?
  • Worden de juiste Wicket-objecten gebruikt voor subcomponenten i.p.v. dat subcomponenten opnieuw worden geïmplementeerd?
    • Als het component BEM-elementen bevat, krijgen deze elementen hun markup dan van hun eigenaar of van een andere component dat een kind is van hun eigenaar?
  • Staat de nullability van alle typen goed? Let vooral op Wicket-modellen voor gebruikersinvoer: deze moeten vaak null toestaan.
  • onComponentTag(...):
    • Controle op de juiste tagnaam moet voor de aanroep super.onComponentTag(tag);.
    • Worden attributen alleen open- en open-close-tags gezet (!tag.isClose())? De gezette attributen worden niet weggeschreven door Wicket, maar het is zonde van de onnodige verwerking.
  • Worden de juiste namen voor waarden van CSS-enums gebruikt?
  • Is er een voorbeeldpagina die de HTML- en Java-code, en die als het mogelijk is het component toont, met daarbij instellingen op de parameters aan te passen.
  • Legt de Javadoc uit hoe het component moet worden gebruikt?
    • Het is niet nodig om langere codesnippets op te nemen: ga ervan uit dat de ontwikkelaar Wicket-kennis heeft.
    • Aan welke tag worden de componenten gehangen?
    • Welke factorymethodes zijn er nodig?
    • Bij een border: waar komt de content terecht?
    • Welke modellen en behaviors kun je meegeven om de weergave te sturen?
    • Welke andere opties zijn er om de weergave aan te passen?
    • Links naar gerelateerde Java-types.
    • Link naar NLDS-component (als dat bestaat).
    • Link naar de community-implementatie op basis waarvan het component is gebouwd.
    • Link naar externe documentatie met aan te passen voorbeeld(en) en uitgebreidere codevoorbeelden (HTML en Java).
  • Zijn er voldoende tests van goede kwaliteit?

@keesvandieren
Copy link
Contributor

@jstuyts kan je voorbeelden toevoegen zodat we kunnen zien hoe het uitpakt als je er mee werkt?

Kees van Dieren added 3 commits December 19, 2025 12:10
We need to use gap with flexbox layouts
We need to use gap with flexbox layouts
@sonarqubecloud
Copy link

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Checkbox Group-component voor Apache Wicket

4 participants