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.

Sign up for GitHub

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

Jump to bottom

BundesSans kann aus unserem Repo entfernt werden #6763

Closed
deleonio opened this issue Aug 30, 2024 · 8 comments · Fixed by #6948
Closed

BundesSans kann aus unserem Repo entfernt werden #6763

deleonio opened this issue Aug 30, 2024 · 8 comments · Fixed by #6948
Assignees
@AlexanderSchmidtCE

Comments

@deleonio
Copy link
Contributor

deleonio commented Aug 30, 2024 •
edited by laske185
Loading

Als Entwickler/Administrator,

möchte ich die Schriftart "BundesSans" aus dem System entfernen,

damit das System schlanker und wartungsfreundlicher wird, da kein Theme mehr auf diese Schriftart zurückgreift.

Betrift sowohl BundesSans als auch BundesSerif.

Akzeptanzkriterien:

  1. "BundesSans" ist vollständig aus dem System entfernt, inklusive aller Referenzen in den Codebasen und Konfigurationsdateien.
  2. Alle Themes und Stylesheets funktionieren weiterhin einwandfrei ohne die "BundesSans"-Schriftart.
  3. Es gibt keine Abhängigkeiten oder Fehler aufgrund des Entfernens der Schriftart.
  4. Das System wird nach der Entfernung getestet, um sicherzustellen, dass die User Experience und das Design nicht beeinträchtigt sind.
  5. Falls BundesSans verwendet wird, soll auf Verdana gewechselt werden. Weitere Fallbacks sind: Arial, Calibri, Helvetica
  6. BundesSerif wird mit Times New Roman ersetzt.

Technische Hinweise:

  • Überprüfung der verwendeten Schriftarten in allen Themes und Stylesheets.
  • Bereinigung von CSS-Dateien, um eventuelle Verweise auf "BundesSans" zu entfernen.
  • Testen der Benutzeroberfläche nach der Entfernung, um sicherzustellen, dass keine visuellen Probleme auftreten.
@Chrisdo82
Copy link
Contributor

Im BMF Theme wird die BundesSans genutzt, wenn ihr diese entfernt, dann wird nur noch der Fallback greifen. Bitte daher erst entfernen, sobald wir das Theme zu uns rübergeholt haben.

@deleonio deleonio moved this from Backlog to 🕒 Waiting in KoliBri Board Sep 6, 2024
@deleonio
Copy link
Contributor Author

deleonio commented Sep 6, 2024

@Chrisdo82 bitte priorisiert den Umzug, damit wir das Thema nicht zu weit hinausschieben müssen. Danke!

@Chrisdo82
Copy link
Contributor

@deleonio wird gemacht. Nochmal fürs Verständnis. Folgende Schritte müssen auf unserer Seite dazu getan werden:

  1. https://github.com/public-ui/kolibri-theme-bwst kopieren
  2. alles unter src mit den Daten unseres Themes austauschen
  3. bswt durch unseren Theme Namen ersetzen
  4. Anweisungen in der Readme befolgen
  5. deployment in die Registry

So in etwa korrekt?

@Chrisdo82
Copy link
Contributor

@deleonio der Umzug ist erledigt. Ihr könnt sowohl das Theme, als auch die Schrift entfernen

@deleonio
Copy link
Contributor Author

Hi @Chrisdo82,

können wir das BMF Theme von Euch installieren?

@deleonio deleonio moved this from 🕒 Waiting to Backlog in KoliBri Board Oct 16, 2024
@deleonio deleonio removed the status in KoliBri Board Oct 16, 2024
@deleonio deleonio moved this to Backlog in KoliBri Board Oct 16, 2024
@Chrisdo82
Copy link
Contributor

@deleonio

Nein, soweit sind wir noch nicht. Wir haben es weder als eigenes Package aufgebaut noch es frei verfügbar gemacht. Es ist jetzt Teil unserer internen Lib.

@laske185
Copy link
Contributor

Welche alternative Schriftart soll verwendet werden?

@laske185 laske185 mentioned this issue Oct 17, 2024
Closed
@laske185 laske185 moved this from Backlog to 📄 Ready in KoliBri Board Oct 17, 2024
@AlexanderSchmidtCE AlexanderSchmidtCE self-assigned this Oct 21, 2024
AlexanderSchmidtCE added a commit that referenced this issue Oct 21, 2024
@AlexanderSchmidtCE
removed BundesSans
152974c 
Refs: #6763
AlexanderSchmidtCE added a commit that referenced this issue Oct 21, 2024
@AlexanderSchmidtCE
Changed font-family-serif
771056d 
Refs: #6763
AlexanderSchmidtCE added a commit that referenced this issue Oct 21, 2024
@AlexanderSchmidtCE
Changed order of replaced fonts
96df8ee 
Refs: #6763
AlexanderSchmidtCE added a commit that referenced this issue Oct 21, 2024
@AlexanderSchmidtCE
formating
21c8d44 
Refs: #6763
AlexanderSchmidtCE added a commit that referenced this issue Oct 21, 2024
@AlexanderSchmidtCE
revert remove of BundesSerif
1f0de70 
Refs: #6763
@AlexanderSchmidtCE AlexanderSchmidtCE linked a pull request Oct 21, 2024 that will close this issue
Removed BundesSans Font #6948
Merged
publicuibot bot pushed a commit that referenced this issue Oct 22, 2024
@AlexanderSchmidtCE @publicuibot
Update all snapshots$
ef02527 
$
Refs: #6763
publicuibot bot pushed a commit that referenced this issue Oct 22, 2024
@AlexanderSchmidtCE @publicuibot
Update all snapshots$
83660dc 
$
Refs: #6763
AlexanderSchmidtCE added a commit that referenced this issue Oct 22, 2024
@AlexanderSchmidtCE
Merge branch 'develop' into 6763-bundessans-kann-aus-unserem-repo-ent…
2888121 
…fernt-werden

# Conflicts:
#	packages/test-tag-name-transformer/snapshots/theme-default/snapshot-for-handout-basic-firefox-linux.png
#	packages/test-tag-name-transformer/snapshots/theme-default/snapshot-for-scenarios-focus-elements-component-accordion-firefox-linux.png
#	packages/test-tag-name-transformer/snapshots/theme-default/snapshot-for-select-basic-firefox-linux.png
#	packages/themes/bmf/snapshots/theme-bmf/snapshot-for-badge-basic-firefox-linux.png
#	packages/themes/bmf/snapshots/theme-bmf/snapshot-for-badge-button-firefox-linux.png
#	packages/themes/bmf/snapshots/theme-bmf/snapshot-for-heading-badge-firefox-linux.png
#	packages/themes/bmf/snapshots/theme-bmf/snapshot-for-version-context-firefox-linux.png

Refs: #6763
publicuibot bot pushed a commit that referenced this issue Oct 22, 2024
@AlexanderSchmidtCE @publicuibot
Update all snapshots$
bc41042 
$
Refs: #6763
@github-project-automation github-project-automation bot moved this from 🔍 Review to ✅ Done in KoliBri Board Oct 23, 2024
sdvg added a commit that referenced this issue Oct 23, 2024
@sdvg
Merge branch 'develop' of github.com:public-ui/kolibri into feature/6…
f8c672f 
…661-git-hooks-precommit

* 'develop' of github.com:public-ui/kolibri: (102 commits)
  Update all snapshots$ $ Refs: #6763
  Update all snapshots$ $ Refs: #6763
  Update all snapshots$ $ Refs: #6763
  Fix Readme markdown format and tweak punctuation
  formating
  Changed order of replaced fonts
  Changed font-family-serif
  removed label from icon
  removed BundesSans
  added sample for empty table
  added build label for Icon
  test fix
  format
  updated readme.md
  added ariablabel and label to icon of alertTitle
  Update all snapshots$ $ Refs: #6815
  Remove duplicate snapshot tests for roast component
  Update all snapshots$ $ Refs: #6400
  Add screenshot test for kol-toast
  Move test and update description
  ...
sdvg added a commit that referenced this issue Oct 24, 2024
@sdvg
Merge branch 'develop' of github.com:public-ui/kolibri into 6343-dom-…
7357e39 
…events-without-deco

* 'develop' of github.com:public-ui/kolibri: (59 commits)
  Update all snapshots$ $ Refs: #6909
  Revert "Update all snapshots$"
  Update all snapshots$ $ Refs: #6909
  Revert "Update all snapshots$"
  Update all snapshots$ $ Refs: #6909
  Revert "Update all snapshots$"
  fix
  Update all snapshots$ $ Refs: #6763
  Update all snapshots$ $ Refs: #6909
  fix merging
  Revert "Update all snapshots$"
  refactoring
  Update all snapshots$ $ Refs: #6763
  Update all snapshots$ $ Refs: #6763
  Fix Readme markdown format and tweak punctuation
  Update all snapshots$ $ Refs: #6909
  formating
  Changed order of replaced fonts
  Changed font-family-serif
  removed label from icon
  ...
@Supportic
Copy link

Supportic commented Nov 14, 2024 •
edited
Loading

Schriftart aus "Erste Schritte" docs entfernen: https://public-ui.github.io/docs/get-started/first-steps#einbinden-von-schriftarten
<link rel="stylesheet" href="assets/bundes/style.css" />

@deleonio deleonio mentioned this issue Nov 15, 2024
Open
@deleonio deleonio mentioned this issue Dec 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlexanderSchmidtCE AlexanderSchmidtCE

Labels
None yet
Projects
KoliBri Board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed BundesSans Font public-ui/kolibri
5 participants
@deleonio @Chrisdo82 @Supportic @laske185 @AlexanderSchmidtCE