fix: Charakter-Modal ohne Scrollbalken trotz größerem Portrait #19

Merged

DasAoD merged 1 commit from fix/character-modal-height into main

2026-08-03 22:57:49 +02:00

DasAoD commented

2026-08-03 22:41:07 +02:00

Owner

Problem

Das Portrait im Charakter-Modal ist mit 160px deutlich größer als der vorherige Buchstaben-Platzhalter (100px, siehe PR #18). Dadurch wurde der Modal-Inhalt auf üblichen Bildschirmhöhen zu hoch und brauchte einen internen Scrollbalken.

Rückmeldung nach dem Deploy von PR #18 (Live-Test mit echten Charakteren): Portrait größer als Platzhalter → alles darunter rutscht nach unten → Scrollbalken im Modal.

Änderungen

Waffen- und Tränke-Zeile im Charakter-Modal tauschen die Plätze (guild.js): Waffen jetzt direkt unter Ehre/Rang, Tränke-Sektion darunter.
.modal-content darf bis 95vh statt 80vh wachsen (guild.css).
Etwas weniger Padding oberhalb von Name/Schließen-Button im Modal-Header sowie am unteren Rand der letzten Attribute-Zeile (guild.css), um zusätzlichen Platz zu gewinnen.
Neue, auf das Charakter-Modal beschränkte Regel #characterModalBody { padding-top: ... } (spart etwas Platz, ohne die anderen Modals — Import/Bestätigen — zu beeinflussen).

Alle drei Anpassungen zusammen reichen aus, damit das Charakter-Modal auf gängigen Bildschirmhöhen (getestet bei 958px Viewport-Höhe) ohne internen Scrollbalken auskommt.

Nebenbei: Die CSS-Cache-Version (includes/template.php) war seit dem Portrait-PR nicht gebumpt, obwohl guild.css sich dort geändert hatte (.char-portrait-frame 100px→160px) — mit hochgezogen, damit Browser die aktualisierten Styles auch laden. JS-Cache-Version ebenfalls gebumpt.

Test

Vor dem Öffnen dieser PR live im Browser simuliert (CSS-Overrides + DOM-Reorder direkt auf der Live-Seite injiziert, nicht deployt) und geprüft, dass der Modal-Inhalt ohne Scrollbalken passt. Nach dem Merge/Deploy bitte nochmal am echten Modal gegenprüfen, insb. bei kleineren Bildschirmhöhen.

## Problem Das Portrait im Charakter-Modal ist mit 160px deutlich größer als der vorherige Buchstaben-Platzhalter (100px, siehe PR #18). Dadurch wurde der Modal-Inhalt auf üblichen Bildschirmhöhen zu hoch und brauchte einen internen Scrollbalken. Rückmeldung nach dem Deploy von PR #18 (Live-Test mit echten Charakteren): Portrait größer als Platzhalter → alles darunter rutscht nach unten → Scrollbalken im Modal. ## Änderungen - **Waffen- und Tränke-Zeile im Charakter-Modal tauschen die Plätze** (`guild.js`): Waffen jetzt direkt unter Ehre/Rang, Tränke-Sektion darunter. - `.modal-content` darf bis `95vh` statt `80vh` wachsen (`guild.css`). - Etwas weniger Padding oberhalb von Name/Schließen-Button im Modal-Header sowie am unteren Rand der letzten Attribute-Zeile (`guild.css`), um zusätzlichen Platz zu gewinnen. - Neue, auf das Charakter-Modal beschränkte Regel `#characterModalBody { padding-top: ... }` (spart etwas Platz, ohne die anderen Modals — Import/Bestätigen — zu beeinflussen). Alle drei Anpassungen zusammen reichen aus, damit das Charakter-Modal auf gängigen Bildschirmhöhen (getestet bei 958px Viewport-Höhe) ohne internen Scrollbalken auskommt. Nebenbei: Die CSS-Cache-Version (`includes/template.php`) war seit dem Portrait-PR nicht gebumpt, obwohl `guild.css` sich dort geändert hatte (`.char-portrait-frame` 100px→160px) — mit hochgezogen, damit Browser die aktualisierten Styles auch laden. JS-Cache-Version ebenfalls gebumpt. ## Test Vor dem Öffnen dieser PR live im Browser simuliert (CSS-Overrides + DOM-Reorder direkt auf der Live-Seite injiziert, nicht deployt) und geprüft, dass der Modal-Inhalt ohne Scrollbalken passt. **Nach dem Merge/Deploy bitte nochmal am echten Modal gegenprüfen**, insb. bei kleineren Bildschirmhöhen.

DasAoD added 1 commit

2026-08-03 22:41:07 +02:00

fix: Charakter-Modal ohne Scrollbalken trotz größerem Portrait 964c93ed0a

Das Portrait ist mit 160px deutlich größer als der vorherige
Buchstaben-Platzhalter (100px), wodurch der Modal-Inhalt auf
üblichen Bildschirmhöhen einen internen Scrollbalken brauchte.

- Waffen- und Tränke-Zeile im Charakter-Modal tauschen die Plätze
- modal-content darf bis 95vh statt 80vh wachsen
- etwas weniger Padding oberhalb von Name/Schließen-Button sowie
  am unteren Rand der Attribute-Zeile

Nebenbei: CSS-Cache-Version war seit dem Portrait-PR nicht gebumpt,
obwohl guild.css sich geändert hatte — mit hochgezogen.

DasAoD merged commit d4a7dd6f5d into main

2026-08-03 22:57:49 +02:00

DasAoD referenced this pull request from a commit

2026-08-03 22:57:51 +02:00

Merge pull request 'fix: Charakter-Modal ohne Scrollbalken trotz größerem Portrait' (#19) from fix/character-modal-height into main

DasAoD referenced this pull request

2026-08-03 23:06:39 +02:00

fix: Waffen-Reihe auf Höhe von Füße/Schmuck ausrichten #20