Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🐞 Bug: 11.7 Schriftgröße wird bei Vergrößerung nicht übernommen #5517

Closed
fohi17 opened this issue Nov 3, 2023 · 5 comments · Fixed by #6187 or #6185
Closed

🐞 Bug: 11.7 Schriftgröße wird bei Vergrößerung nicht übernommen #5517

fohi17 opened this issue Nov 3, 2023 · 5 comments · Fixed by #6187 or #6185
Assignees
Labels

Comments

@fohi17
Copy link

fohi17 commented Nov 3, 2023

Note

  • [x ] Ich achte beim Erstellen des Issues darauf, dass ich keine fachlichen und unternehmensinternen Details preisgebe.

Schriftgröße wird bei Vergrößerung nicht übernommen

Beschreibung des Fehlers

Wenn man die Schriftgröße im Browser größer stellt (gemäß BIK BITV 11.7), wird das bei mehreren Komponenten für Überschriften, Links und Buttons (siehe unten) nicht übernommen, und die Schrift bleibt klein.
Die Ursache ist wahrscheinlich die font-size, die für die Elemente über var auf 16px festgelegt ist.

Reproduktion

Style: BMF, Z2
Version: v2.0.0-rc.6

Schritte zum Reproduzieren des Verhaltens:

Schriftgröße des Browsers ändern

Im Chrome:
Einstellungen -> Darstellung -> Schriftgröße
Im Firefox:
Einstellungen -> Allgemein -> Schriftarten

Prüfe folgende Komponenten:

BMF: Überschriften

  • alert
  • card

BMF: Links

  • nav

Zoll(v2): Alle Links in allen Komponenten
bspw.: breadcrumb, link-group

Zoll(v2): Alle Buttons in allen Komponenten
bspw.: accordion, button-group, button etc.

Erwartetes Verhalten

Der Text von allen Texten, inkl. Überschriften, Links und Buttons wird mitskaliert.

Screenshots

image
image

@fohi17 fohi17 added the bug Something isn't working label Nov 3, 2023
@deleonio
Copy link
Contributor

deleonio commented Nov 3, 2023

Das Styling ist gekapselt, wie man bei der Headline sieht. Wir prüfen, die inherit Option.

@deleonio
Copy link
Contributor

deleonio commented Feb 8, 2024

@sdvg: Bitte mal eine Lösungsskizze erstellen.

@deleonio deleonio moved this to 📄 Ready in KoliBri Board Feb 8, 2024
@sdvg sdvg moved this from 📄 Ready to 🏗 In progress in KoliBri Board Feb 9, 2024
@sdvg
Copy link
Member

sdvg commented Feb 9, 2024

Ich würde folgende Konventionen für Größeneinheiten in CSS vorschlagen:

  • Allgemein wird rem verwendet
  • Ausnahmen:
    • px für: border-width, border-radius, box-shadow (Diese skalieren nicht gut, und müssen es auch nicht)
    • Relative Einheiten (%, vw, etc.) können verwendet werden, wo sie Sinn ergeben (z.B. Layout)
  • Um mit verschiedenen root font sizes in verschieden Projekten umgehen zu können:
    • Definiert das Projekt optional eine CSS custom property (--kolibri-root-font-size: 10)
    • Verwenden wir bei der Benutzung von rem grundsätzlich eine Sass function.
    • Optional: Außerdem sprechen wir für neue Projekt die Empfehlung aus, die root font-size auf 62.5% zu setzen. Das entspricht 1rem = 10px und macht die Arbeit z.B. in den Dev Tools am einfachsten.
// Sass Funktion, wie sie in KoliBri genutzt wird:
@function rem($size) {
  @return calc(#{$size}rem / var(--kolibri-root-font-size, 16));
}

// Optionale Projekt-Konfiguration, wenn root font-size != 16px
html {
  font-size: 62.5%;
  --kolibri-root-font-size: 10;
  // wird beides weggelassen, gehen wir von einer Standardgröße von 16px aus -> funktioniert ebenfalls.
}

// Hier einmal exemplarisch verwendet. Erzeugt `calc(16rem / var(--kolibri-root-font-size, 16))`, was computed 1.6rem bzw. 16px entspricht:
body {
  font-size: rem(16);
}

@deleonio
Copy link
Contributor

Das machen wir so.

@sdvg
Copy link
Member

sdvg commented Mar 11, 2024

Die beschriebenen Fehler sind im v1-Theme vom BMF (das wir nicht mehr weiter pflegen) noch reproduzierbar, in v2 aber schon gelöst ✅

@sdvg sdvg moved this from 🏗 In progress to 🔍 Review in KoliBri Board Mar 11, 2024
@github-project-automation github-project-automation bot moved this from 🔍 Review to ✅ Done in KoliBri Board Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment