detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
43 stars 13 forks source link

Readonly styling virker ikke i firefox #14

Closed postergaard closed 5 years ago

postergaard commented 5 years ago

Bekskriv fejlen (Describe the bug) Styling til readonly felter, slår ikke igennem i firefox

Genskab fejlen (To Reproduce) Sådan genskabes fejlen (Steps to reproduce the behavior):

  1. Åben en firefox browser
  2. Gå til et readonly felt

Indsæt link (Add a URL) https://detfaellesdesignsystem.github.io/dkfds-docs/komponenter/form-controls/#felter

Mulig løsning https://www.w3schools.com/cssref/sel_read-only.asp

Screenshots Her er styling image

Her virker det (chrome) image

Her virker det ikke (FireFox) image

Sådan burde det fungere (Expected behavior) Read only skal defineres specifikt for firefox ex:

input:-moz-read-only { / For Firefox / / readonly styling/ }

Desktop - udfyld venligst følgende (please complete the following information):

detfaellesdesignsystem commented 5 years ago

Hej @postergaard,

Tak for din henvendelse.

Vi har oprettet en sag på det, og vil vende tilbage så snart vi har kigget på det.

I mellemtiden - kan jeg da få dig til at prøve at kigge på https://detfaellesdesignsystem.github.io/dkfds-docs/komponenter/form-controls/ i firefox, og se om read-only eksemplet heller ikke virker der? - jeg har nemlig ikke kunnet reproducere det i min ende.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

postergaard commented 5 years ago

Hej Kristina

Når jeg kigger på koden i browseren, kan jeg se at I her har indsat det specifikt til firefox:

image

Men det er der ikke gjort i koden:

https://github.com/detfaellesdesignsystem/dkfds-components/blob/f842105f0d58e7c6f1caf9c4d908da8044dbf904/src/stylesheets/elements/_inputs.scss#L145

Har lavet en pull request her: https://github.com/detfaellesdesignsystem/dkfds-components/pull/15

detfaellesdesignsystem commented 5 years ago

Hej @postergaard,

Jeg har nu kigget nærmere på det.

Det viser sig, at hvis man vil bruge scss filerne og ikke css filerne i sit projekt, skal man selv sørge for at køre scss filen igennem autoprefixer.

Jeg har tilføjet dokumentation omkring dette under Til udviklere på dokumentationssitet.

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

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

postergaard commented 5 years ago

Mange tak for hurtigt svar. Jeg lukker sagen