66 lines
3.7 KiB
Markdown
66 lines
3.7 KiB
Markdown
# Integration der Ernährungsseite
|
|
|
|
Review-Branch: `integration/ernaehrung-review`, erstellt aus `main`.
|
|
Die vier Quellbranches wurden mit Merge-Commits eingebunden. Ihre Historie
|
|
bleibt erhalten; die Quellbranches und `main` wurden nicht verändert.
|
|
|
|
| Bereich | Quellbranch / Stand | CSS-Datei | CSS-Bereich |
|
|
| --- | --- | --- | --- |
|
|
| Shahnam | `S-Produktübersicht` / `f7605d8` | `css/produktuebersicht.css` | `.produktuebersicht` |
|
|
| Vincent | `feature/V-US-01-02-leistungsuebersicht` / `312e788` | `css/leistungen.css` | `.pakete` |
|
|
| Ahmad | `A-Berater-und-Qualifikationen` / `d945d0a` | `css/berater.css` | `.berater-section` |
|
|
| Dennis | `D-info-ueber-beratung` / `6b2bcb4` | `css/beratung.css` | `.beratung-bereich` |
|
|
|
|
`css/site.css` enthält nur die gemeinsame Seitengrundlage. Gleiche Dateinamen
|
|
an unterschiedlichen Pfaden wären technisch erlaubt. Gegen Stilkonflikte
|
|
helfen jedoch die begrenzten Selektoren und Variablen, nicht der Dateiname.
|
|
Beispiel: `.produktuebersicht .container` betrifft nur Shahnam.
|
|
Neue Regeln bitte innerhalb des jeweiligen CSS-Bereichs halten.
|
|
|
|
## Aufgelöste Konflikte und Fehler
|
|
|
|
- `index.html`: ein Dokument mit einem `main`; alle vier Beiträge erhalten.
|
|
- Gleichnamige CSS-Dateien: getrennte Namen, lokale Variablen und Selektoren.
|
|
- Dennis' `style.css` / `script.js`: Modify/Delete-Konflikte durch Übernahme
|
|
nach `css/beratung.css` / `js/beratung.js` aufgelöst.
|
|
- Pakete: falscher JSON-Pfad von `data/` auf `json/` korrigiert;
|
|
Fehleranzeige ergänzt. Die in Vincents erstem Commit vorhandene Auswahl
|
|
samt Event `paket:ausgewaehlt` funktioniert wieder mit den JSON-Karten.
|
|
- Berater: Kartenbreite für kleine Displays begrenzt; JS nur von überflüssigen
|
|
Leerzeichen bereinigt, Daten und Rendering-Logik unverändert.
|
|
- Beratung: Hintergrund und Scroll-Reveal auf den eigenen Bereich begrenzt;
|
|
Grid und lange Wörter verursachen auf kleinen Displays keinen Überlauf.
|
|
- Schriftarten einmal im gemeinsamen Head eingebunden; `.DS_Store` entfernt.
|
|
|
|
JSON-Inhalte, Bilder, Preise, Links und Texte wurden nicht redaktionell geändert.
|
|
Zur fachlichen Prüfung bleiben unter anderem die zwei gleich benannten
|
|
Beraterprofile, die Paketbezeichnung `STANDART`, Platzhalter und Buchungslinks.
|
|
Die Paketauswahl ist noch keine Buchungsfunktion.
|
|
|
|
## Lokal prüfen
|
|
|
|
`index.html` über VS Code Live Server öffnen, nicht direkt als `file://`;
|
|
die beiden JSON-Dateien werden per `fetch` geladen.
|
|
|
|
Geprüft in Chrome bei 320, 390, 768, 1366 und 1920 px: alle vier Bereiche,
|
|
3 Pakete, 2 Berater, Bilder, eindeutige IDs und kein horizontaler Überlauf.
|
|
Zusätzlich: Paketwechsel mit genau einer Auswahl und korrektem Event,
|
|
reduzierte Bewegung, Fehleranzeigen bei fehlenden JSON-Dateien sowie keine
|
|
unbehandelten JavaScript-Fehler oder fehlgeschlagenen lokalen Ressourcen im Normalbetrieb.
|
|
|
|
## Weitere Änderungen zusammenführen
|
|
|
|
1. Auf diesen Integrationsbranch wechseln und `git fetch origin` ausführen.
|
|
2. Den aktualisierten Quellbranch mit `git merge origin/BRANCHNAME` einbinden.
|
|
3. Bei Konflikten mit `git status` die betroffenen Dateien prüfen.
|
|
`<<<<<<<`, `=======` und `>>>>>>>` markieren die beiden Versionen.
|
|
In der gemeinsamen HTML-Datei beide benötigten Bereiche zusammensetzen;
|
|
nicht pauschal eine komplette Seite mit „ours“ oder „theirs“ ersetzen.
|
|
4. Marker entfernen, Pfade und CSS-Bereiche prüfen und im Browser testen.
|
|
5. Aufgelöste Dateien mit `git add DATEIPFAD` markieren, `git commit` ausführen
|
|
und ausschließlich diesen Integrationsbranch pushen.
|
|
|
|
Ein noch laufender Merge lässt sich mit `git merge --abort` abbrechen.
|
|
Bereits abgeschlossene Merges bleiben bestehen. Erst nach Team-Review soll
|
|
ein Pull Request von diesem Branch nach `main` zusammengeführt werden.
|