search

detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
43 stars 13 forks source link

Modal "hide" funktion checker ikke om backdrop eksisterer inden den fjernes #210

Open hanshvidberg opened 1 year ago

hanshvidberg commented 1 year ago

Som aftager af DKFDS kan det ske at åbne/lukke-funktionaliteten af modals bliver styret udefra, og der kan godt opstå en konflikt med 'keyup' event-handleren som lukker modalen på escape, uden man kan gøre noget ved det. Hvis man efter at modalen er blevet lukket med escape, har brug for at kalde Modal.hide() programmatisk, så checker funktionen ikke om backdrop allerede er fjernet, og dermed er resultatet en hård js fejl i browseren.

Kunne løses med et enkelt if ($backdrop) inden det bliver forsøgt fjernet fra DOM'en.

https://github.com/detfaellesdesignsystem/dkfds-components/blob/edf45da31b93deddf6ec32d501729847456ade02/src/js/components/modal.js#L40-L41

detfaellesdesignsystem commented 1 year ago

Hej @hanshvidberg

Tak for din henvendelse!

Vi har oprettet en sag på det.

Med venlig hilsen Helle Juhl Det Fælles Designsystem

detfaellesdesignsystem commented 1 year ago

Hej @hanshvidberg

Fejlen er blevet rettet i version 9.0.1.

Med venlig hilsen Helle Juhl Det Fælles Designsystem