Skip to content

feat!: add forwardref to links

Jakob Kielland requested to merge jakie/feat-add-forwardref-to-links into main

Wrapper Link, TabLink og ButtonLink i React.forwardRef() så de kan brukes med blant annet Next.js-links. React anbefaler at dette markeres som en breaking change, så jeg har gjort dette i commit-meldingene.

Har skrudd av squash-on-merge for denne, siden det skal være tydelig at hver av changesa er breaking for hver sin pakke.

Første gang jeg bruker forwardRef, så jeg har testet både at ting funker som før og at det nå funker å passe en ref til komponentene. Både manuelt vha. useRef og gjennom komponenter som Next.js sin Link.

PS: Ser ut som det er veldig mange endrede linjer, men er egentlig ikke det. Kommer av at indentation endra seg siden komponentene fikk en wrapper.

Edited by Jakob Kielland

Merge request reports

Loading