search

detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
44 stars 13 forks source link

Modal printes 2 gange #46

Closed FNA-atp closed 5 years ago

FNA-atp commented 5 years ago

Hej dk-fds, Ved print (ctrl+p) udskrives en modal 2 gange. Det kan testes her: https://designsystem.dk/pages/eksempler/aktindsigt/aktindsigt-1/ image

Har I mulighed for at se på denne fejl? :-)

Vh. Farzana

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp,

Tak for din henvendelse!

Vi har oprettet en sag på dette, og vil kigge nærmere på det.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

FNA-atp commented 5 years ago

Hej igen, Når man printer modalen ud på IE - Side 1: Modalens tekst ligger oven i den tekst, der ligger under modalen, hvilket gør teksten helt ulæselig. Side 2: Udprintet er pænt og læsbart.

Side 1: image

Side 2: image

Har I mulighed for at ekskludere side 1 på IE?

Vh. Farzana

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp

Vi har nu implementeret en løsning, som vil komme med i næste release.

Sidens indhold bliver kort sagt helt skjult, når en modal vises, så man undgår dette issue med flere sider og dermed duplikater af modalen.

Sig endelig til, hvis du har spørgsmål.

Jeg lukker dog sagen her - men du er velkommen til at genåbne den.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

FNA-atp commented 5 years ago

Hej Kristina, Det lyder godt - Tak for det :-) Mvh. Farzana

FNA-atp commented 5 years ago

Hej igen, Det ser ud til at fejlen stadig er der, selvom vi har opdateret til 4.4.0. Modalen udskrives de antal gange/sider, som siden i baggrunden har.

Skal vi gøre andet end at opdatere versionen for, at release-ændringer bliver slået igennem?

Mvh. Farzana

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp,

Måden det bla. er blevet løst på, er at der bliver sat en klasse på body modal-active. Denne klasse gør at alt på siden - på nær modal, bliver skjult ved print, og siden derfor ikke fylder mere end nødvendigt.

Er du sikker på, at I henter 4.4.0 versionen af dkfds.js ind i projektet?

Evt. tjek at klassen bliver sat, når jeres modal er aktiv.

I hvilken forbindelse er jeres modal sat ind på siden?

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

clpatp commented 5 years ago

Når jeg lige tester så bliver der sat modal-open på body - ikke modal-active ?

Og uddrag fra package.json :

    "dkfds": "^4.4.0",
    "dkfds-plugins": "^4.4.0",
detfaellesdesignsystem commented 5 years ago

Hej @clpatp ,

modal-open er ikke en klasse der kommer fra vores løsning. Det er dog den klasse bootstrap sætter på body, når deres modal er åben. Jeg vil anbefale, at du kigger løsningen igennem.

Om det lige er bootstrap der gør sig gældende her, skal jeg ikke kunne sige. Men det er i hvert fald noget lignende de gør.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

clpatp commented 5 years ago

Hej Kristina

vi kigger på tingene igen og vender tilbage hvis det bliver nødvendigt :)

FNA-atp commented 5 years ago

Hej Kristina, Vi har nu kigget på denne igen. Den løsning(vha. javascript), som I har lavet, fungerer ikke hos os, fordi vi arbejder kun med Typescript og angular.

Er det muligt for jer at lave en anden løsning? eller skal vi lave en workaround hos os?

Vh. Farzana

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp ,

Hvad er det der gør at den ikke virker hos jer?

Jeg sidder faktisk lige nu og er ved at tilpasse modal komponenten. Så hvis jeg ved nærmere, hvad det er der gør, at I ikke kan bruge den nuværende, så kan jeg måske gøre noget ved det nu her.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

FNA-atp commented 5 years ago

Hej Kristina, Hvis vi kun indsætter classe= "modal-active" ind på , så skjules alt andet indhold på siden ved print. image image

Når modallen er åben: image

Her skjules alt andet indhold på siden, som den skal:

image

Her har vi ikke sat "modal-active" ind på : image image

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp ,

Jeg må indrømme, at jeg ikke forstår dit svar.

I forbindelse med dit svar: "Den løsning(vha. javascript), som I har lavet, fungerer ikke hos os, fordi vi arbejder kun med Typescript og angular." spørger jeg ind til, hvad det helt præcist er der ikke fungerer hos jer?

modal-active sættes på body elementet, når en modal er aktiv. Dette for at skjule det bagvedliggende indhold på siden og derved undgå at der printes unødigt ekstra sider samt at modalen ikke printes oven på andet tekst.

Sig endelig til, hvis noget af det jeg skriver ikke giver mening.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

FNA-atp commented 5 years ago

Hej igen :-) Den logik, som I har lavet med javascript(jquery), hvor "modal-active" sættes på, når en modal er åben og tages ned igen, når den ikke er aktiv/åben - Det virker ikke hos os - fordi vi arbejder/bruger ikke jeres javascript. Vi bruger kun html/css-styling fra DFS.

Hvis vi skal få "modal-active" til at virke hos os, skal vi selv lave logikken hos os og lave en workaround, hver gang vi bruger en modal i en løsning. Medmindre I kan løse problemet vha. CSS/Sass.

Håber ovenstående giver mening :-)

detfaellesdesignsystem commented 5 years ago

Hej @FNA-atp,

Javascripten i FDS er en vigtig del af at det hele virker. Vi kan ikke tage højde for projekter, som kun vælger af bruge halvdelen af dkfds desværre.

Jeg har dog lagt en sag i backloggen om at vi bør kigge på, hvordan vi kan gøre javascripten tilgængelig for typescript/angular projekter. Hvis du har en idé her, så hold dig endelig ikke tilbage.

Da I ikke bruger dkfds javasvcripten, så skal I selv sørge for at replikere samme opførsel i jeres eget projekt.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

FNA-atp commented 5 years ago

Hej Kristina, Det er ikke kun i et projekt , men alle vores løsninger/projekter i ATP er bygget sådan, desværre :-/ Det kunne være fedt, hvis I kunne tage det med i jeres overvejelser, når nogle issues skal løses vha. javascripten.

Håber, I kan finde en løsning på, hvordan vi kan bruge javascripten i vores projekter. Jeg skal melde tilbage, hvis vi støder på en løsning :-)

Vh. Farzana