detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
43 stars 13 forks source link

Tabs/Faneblad: ERROR TypeError: Cannot read properties of undefined (reading 'parentNode') #192

Open Flaeng opened 1 year ago

Flaeng commented 1 year ago

Beskriv fejlen (Describe the bug) Hvis man sætter en faneblad op uden at have en aktiv fane får man en 'Cannot read properties of undefined (reading 'parentNode')'

Vær klar og præcis i din beskrivelse af fejlen. (A clear and concise description of what the bug is.)

Genskab fejlen (To Reproduce)

<div class="tabnav" role="tablist">
    <button aria-controls="tabpanel1" id="tab1" class="tabnav-item"
        role="tab" aria-selected="false">
        <span>Fane 1</span>
    </button>
    <section class="tabnav-panel" aria-hidden="true" role="tabpanel"
        tabindex="0" id="tabpanel1" aria-labelledby="tab1">
        <h2>Indhold 1</h2>
        <p>...</p>
    </section>
</div>

Sådan burde det fungere (Expected behavior) Linjen herunder er et null-tjek der er vendt forkert. På linjen under tager man det første element i array'et, men man har lige tjekket om array'et er tomt. https://github.com/detfaellesdesignsystem/dkfds-components/blob/master/src/js/components/tabnav.js#L53

detfaellesdesignsystem commented 1 year ago

Hej @Flaeng

Tak for din henvendelse!

Der skal altid være en aktiv fane i en fanebladsgruppe - har du mulighed for at fortælle noget mere om jeres scenarie, siden I ikke ønsker dette?

Med venlig hilsen Helle Juhl Det Fælles Designsystem

Flaeng commented 1 year ago

Hej Helle Jeg forventede at hvis ingen fane var markeret som aktiv at den ville tage den første fane. Og det ser også ud til at I har forsøgt at gøre dette i koden men blot er kommet til at tjekke på om et array er tomt i stedet for at tjekke på om det ikke tomt.

Om ikke andet burde den vel vise fane-knapperne og så intet fane-indhold under eller give en fejlbesked der er mere sigende end "Cannot read properties of undefined (reading 'parentNode')".

Mvh Dennis

detfaellesdesignsystem commented 1 year ago

Hej @Flaeng

Tak for uddybningen! Jeg har oprettet en sag med din observation.

Med venlig hilsen Helle Juhl Det Fælles Designsystem