detfaellesdesignsystem / dkfds-components

Styling og komponenter
Other
43 stars 13 forks source link

Print - style og utilities #26

Closed laclogin closed 5 years ago

laclogin commented 5 years ago

Pkt. 1) Print visning Er det meningen af print skal vises som en mobilskærm ? Således at login oplysninger er skjult i menu knappen, eller at tabellerne bliver printet som mobil visning ? (se nedenstående billede) Vi ønsker helst at style print vises som desktop visning, men hvad synes I hos dkfds ?

image image

Pkt. 2) Utilities for toggling display in print
Vi savner at kunne toogle elementer til print. Har I mulighed for at tilføje nedenstående print utilities til dkfds ?

// Utilities for toggling display in print

@media print {
  .d-print-none         { display: none !important; }
  .d-print-inline       { display: inline !important; }
  .d-print-inline-block { display: inline-block !important; }
  .d-print-block        { display: block !important; }
  .d-print-table        { display: table !important; }
  .d-print-table-row    { display: table-row !important; }
  .d-print-table-cell   { display: table-cell !important; }
  .d-print-flex         { display: flex !important; }
  .d-print-inline-flex  { display: inline-flex !important; }
}

Eksempel på en ønskede anvendelse Vores forretning ønsker som standard at, vise login-oplysninger + SE-nr/CVR-nr. på print.

image

image

Henvisning https://getbootstrap.com/docs/4.3/utilities/display/#display-in-print

detfaellesdesignsystem commented 5 years ago

Hej @laclogin ,

Tak for din henvendelse!

Vi har allerede en sag liggende i vores backlog om at vi skal genbesøge print css i dkfds.

Jeg kan ikke love at vi kan nå det inden næste release, men vil sørge for at klasserne du nævner i pkt. 2 kommer med.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

detfaellesdesignsystem commented 5 years ago

Hej @laclogin ,

Print klasserne er nu release med seneste version (4.0.1)

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

laclogin commented 5 years ago

Mange tak for hjælpen.

Er npm 4.0.1 også klar ?
Når jeg installere 4.0.1, får jeg fejlen: npm ERR! notarget No matching version found for dkfds@^4.0.1

Med venlig hilsen Laura Beck ATP

detfaellesdesignsystem commented 5 years ago

Hej @laclogin ,

Ja, 4.0.1 kom ud i går eftermiddag.

Jeg har lige forsøgt at opdatere dkfds versionen i et lokalt projekt jeg har liggende til test - det fungerede fint. Prøv evt. igen?

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

laclogin commented 5 years ago

Hej Kristina,

Jeg prøvede i går eftermiddags og 2 gange i morges at, opdatere til 4.0.1, og jeg fik den samme fejl.

Hvis jeg køre en npm install --save dkfds@latest får jeg version 4.0.0

Med venlig hilsen Laura Beck ATP

detfaellesdesignsystem commented 5 years ago

Hej @laclogin,

Jeg kan simpelthen ikke reproducere den fejl du melder. Jeg er på nippet til at sige det er en projektspecifik fejl.

Med venlig hilsen Kristina Jensen Det Fælles Designsystem

laclogin commented 5 years ago

Hej Kristina, I dag lykkedes det at installere 4.0.1
I går prøvede jeg prøvede at, slette package-lock og node-modules mappen, men fik den samme fejl. Det gjorde min kollega også med hans projekt. Mystisk.

Men alt er vel nu, og tak for hjælpen.

Med venlig hilsen Laura Beck ATP

detfaellesdesignsystem commented 5 years ago

@laclogin Hej Laura Jeg lukker dette issue. Sig endelig til hvis det skal genåbnes. Mvh Sune FDS