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

Apps sollen nicht styles.css der webapp-schulverwaltung verwenden #212

Open
hupf opened this issue Dec 17, 2024 · 0 comments
Open

Apps sollen nicht styles.css der webapp-schulverwaltung verwenden #212

hupf opened this issue Dec 17, 2024 · 0 comments
Labels
tech-task Ein Tech-bezogenes Issue

Comments

@hupf
Copy link
Collaborator

hupf commented Dec 17, 2024

Ausgangslage

Aktuell binden folgende Apps das Haupt-CSS-File der webapp-schulverwaltung ein:

public/apps/Raumreservation/index.html:<link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/Kursausschreibung/index.html:<link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/EmberApps/AnmeldedetailsEinlesen/index.html:<link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/EmberApps/SchulleiterPersonen/index.html:<link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/EmberApps/Fotoliste/index.html:<link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/Noteneingabe/index.html:    <link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">
public/apps/Stellvertretung/index.html:    <link rel="stylesheet"  type="text/css" href="/apps/webapp-schulverwaltung/styles.css">

Dieses «tight-coupling» ist nicht ideal und sollte vermieden werden. Denn falls die Styles der WebApp angepasst werden, die Bootstrap Version aktualisiert wird oder sich das Verhalten beim Angular CLI ändert, betrifft dies auch die Apps. Umgekehrt könnte es zu einem Blocker werden die Web App zu aktualisieren, weil Apps davon betroffen wären.

Es ist zudem nicht klar, auf welche des styles.css sich diese Apps verlassen, man weiss also gar nicht bei welchen Änderungen die Apps betroffen sind.

Lösung

  • Ermitteln was für Styles die Apps verwenden (Fonts, Icons, Bootstrap? Was von Bootstrap?).
  • Dediziertes CSS File erstellen mit einem minimal Set an Base-Styles, welches die Apps als Basis verwenden können.
  • Die Apps dürfen nicht mehr das styles.css der webapp-schulverwaltung einbinden.
  • Bundle Hashing der webapp-schulverwaltung wieder aktivieren: Bundle Hashing aktivieren - 0.5PT webapp-schulverwaltung#706
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech-task Ein Tech-bezogenes Issue
Projects
Status: New
Development

No branches or pull requests

2 participants