Skip to content

Beheer: update document naar rule structuur #12

Merged
TimvdLippe merged 8 commits intodevelopfrom
pr-timvdlippe-rule-structuur
Jun 3, 2025
Merged

Beheer: update document naar rule structuur #12
TimvdLippe merged 8 commits intodevelopfrom
pr-timvdlippe-rule-structuur

Conversation

@TimvdLippe
Copy link
Contributor

@TimvdLippe TimvdLippe commented May 28, 2025

Hiermee gebruiken we dezelfde structuur en styling als het core
document.

Fixes #4

Hiermee gebruiken we dezelfde structuur en styling als het core
document.
@github-actions
Copy link

@TimvdLippe TimvdLippe requested review from lvdbrink and sanderke May 28, 2025 14:21
@TimvdLippe TimvdLippe marked this pull request as ready for review May 28, 2025 14:22
@TimvdLippe
Copy link
Contributor Author

Het zijn veel wijzigingen qua structuur, maar qua inhoud zou er niks gewijzigd moeten zijn. Deze PR zorgt ervoor dat we consistent zijn met de ADR core en dus ook de mooie styling voor regels hebben.

@sanderke Qua styling + functionaliteit zat ik te denken om dit apart te publiceren. Zodat we niet in deze module en in core hetzelfde .css bestandje hoeven te hebben. Tevens leek het mij handig om de functionaliteit wat betreft de "technische" versies "functionele" regels ook ergens neer te zetten. Dan kunnen we de respec configuratie ook delen en hebben we mooie linkjes naar iedere regels. Wat denk jij daar van?

Copy link
Member

@sanderke sanderke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De regels zien er een stuk duidelijker uit zo.
Het importeren i.p.v. kopiëren van scripts en style sheets klinkt als een goed plan.

@TimvdLippe
Copy link
Contributor Author

@lvdbrink Er zijn geen inhoudelijke wijzigingen in deze PR, maar het lijkt me toch handig als jij dat dubbel checkt. Zou je daar deze week nog tijd voor hebben? Mocht je daar geen tijd voor hebben, dan kunnen we het ook mergen met de aanname dat het puur opmaak is.

@TimvdLippe TimvdLippe merged commit aede818 into develop Jun 3, 2025
8 checks passed
@TimvdLippe TimvdLippe deleted the pr-timvdlippe-rule-structuur branch June 3, 2025 12:15
@TimvdLippe TimvdLippe added this to the Geospatial 1.1 milestone Jun 6, 2025
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.

Update regel tekst naar ADR core structuur

3 participants