07th-mod / 07th-mod/higurashi-assembly

Image letterboxing inconsistent between chapters

Offen
#141 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C#
Sterne
12
Forks
12
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### Differences in behavior between chapters

The behavior for letterboxing is different across chapters:

- "Mod" branch + Ch[1, 2]: Left and right black bars only
- In other words, the image height is clamped to the window height, then the width is set to keep the same aspect ratio. Tall images are OK, but images wider than the window will be cut-off
- Console + Ch[3, 4, 5, **7**]: Both Left and right black bars, and top and bottom black bars are supported
- Ch[6, **8**, 9, 10 (hou)] : In addition to both types of letterboxing, ONLY when origin is non-null, special cases are added for specific width and heights:
- A height of 960 is converted to a height of 480
- A width of 1280 is converted to a width of 640
- I don't actually know if this does anything extra compared to normal?

Oddly, chapter 7 is an outlier and doesn't follow the "add new features every N chapters" pattern.

Also, I'm pretty sure we never use non-null origin (originx set to 0), refer to:
- https://github.com/07th-mod/higurashi-assembly/issues/142

### Conclusion

All this taken into account, it more or less means that Chapter 1 and 2 behave differently to all the other chapters with respect to image scaling. This can be easily fixed, but for chapter 1 and 2 this may cause some graphical bugs, if we are accidentally relying on the old behavior.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Das Issue beschreibt inkonsistentes Letterboxing-Verhalten von Bildern über verschiedene Kapitel des Spiels hinweg. Beginnen Sie mit der Untersuchung des Bild-Rendering-Codes, wahrscheinlich in Dateien, die Grafik- oder UI-Skalierung behandeln. Suchen Sie nach bedingter Logik basierend auf Kapitelnummern. Überprüfen Sie das verlinkte Issue #142 für Kontext zur Verwendung von non-null origin. Das Testen von Änderungen erfordert das Ausführen verschiedener Kapitel, um zu überprüfen, dass der Fix keine grafischen Regressionen einführt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp
Bereich
game-dev
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.