Feat/70 Buttons nlds comp

[TessaViergever]

In deze PR

• <Button>, <ButtonGroup>, <Image> toevoegen aan index.ts
• bestaande buttons vervangen voor nlds <Button> Componenten
  • (LocationSelect.tsx als aparte commit omdat Justian hier ook in bezig is, just in case het makkelijker is om deze te droppen dan mergen)
• Buttons in IncidentFormFooter.tsx in een plaatsen
• justify-end verwijderen van IncidentFormFooter.tsx om toegankelijkheid te verbeteren (volgende/terug buttons staan nu onder de andere form onderdelen)
• some refactoring
• Button.tsx verwijderen (ongebruikt)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
signalen-frontend-wcag	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 8, 2024 8:33am

[justiandevs]

Wat mij betreft kan Button.tsx weg. Wat betreft de Button en Image voor de PreviewFile, dat ziet er nu zo uit:

Deze afbeelding hoort terecht te komen in de grid van 5 blokken (zie main build op vercel):

Dus deze verandering breekt nog wel het e.e.a, zou je dit kunnen oplossen met de NLDS componenten?

[TessaViergever]

@justiandevs ik kan niet reageren onder jouw comment dus doe het even zo.

Ik dacht inderdaad dat het over die img grid ging, maar het lukt mij nergens (ook niet in de main build) om daar een foto toe te voegen (en dus vervolgens te verwijderen 🤷🏼‍♀️)?

Als ik de bestaande button en img uit de PreviewFile vervang voor NLDS Button en Image componenten, ziet alles er visueel bij mij hetzelfde uit en lukt het ook niet om een foto toe te voegen dus leek het ook op dat vlak niet anders. 🙃 Toch twijfelde ik of het wel klopte haha dus daarom toch apart gedaan.

Wat is wijsheid? Ik neig naar deze change voor nu even terugdraaien en uit deze PR halen aangezien het een heel ander soort button is (deze werd bijv. ook niet opgehaald uit Button.tsx). Dan kan ik als aparte issue even kijken of ik hier vanuit NLDS iets mee kan. :) Wat vind jij?

[justiandevs]

Dat lijkt mij

@justiandevs ik kan niet reageren onder jouw comment dus doe het even zo.

Ik dacht inderdaad dat het over die img grid ging, maar het lukt mij nergens (ook niet in de main build) om daar een foto toe te voegen (en dus vervolgens te verwijderen 🤷🏼‍♀️)?

Als ik de bestaande button en img uit de PreviewFile vervang voor NLDS Button en Image componenten, ziet alles er visueel bij mij hetzelfde uit en lukt het ook niet om een foto toe te voegen dus leek het ook op dat vlak niet anders. 🙃 Toch twijfelde ik of het wel klopte haha dus daarom toch apart gedaan.

Wat is wijsheid? Ik neig naar deze change voor nu even terugdraaien en uit deze PR halen aangezien het een heel ander soort button is (deze werd bijv. ook niet opgehaald uit Button.tsx). Dan kan ik als aparte issue even kijken of ik hier vanuit NLDS iets mee kan. :) Wat vind jij?

Dat lijkt mij een prima oplossing. Vreemd dat het uploaden niet lukt, kan je dat straks even laten zien?

[TessaViergever]

@justiandevs nvm..

Gister en vanochtend lukte het niet, als ik op + klikte kreeg ik niet eens de optie om foto's te selecteren. Net dacht ik okay nog even 1x proberen voor de zekerheid EN NU DOET IE HET WEL? 🫠 Zowel op local host als op main build. Echt geen idee why, maar hij doet het iig. 🥲 ✨

[justiandevs]

@justiandevs nvm..

Gister en vanochtend lukte het niet, als ik op + klikte kreeg ik niet eens de optie om foto's te selecteren. Net dacht ik okay nog even 1x proberen voor de zekerheid EN NU DOET IE HET WEL? 🫠 Zowel op local host als op main build. Echt geen idee why, maar hij doet het iig. 🥲 ✨

Haha, oke. Soms gebeuren er onverwachte dingen. Gelukkig werkt het weer!

[justiandevs]

Ziet er goed uit! Merge conflicts zijn opgelost. Hierbij dus approved en merged.