open-duelyst/duelyst

[P0] Viewport / screen rotation should be disabled on Mobile

Offen

#221 geöffnet am 29.10.2022

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (526 Forks)batch import
bugfrontendhelp wanted

Repository-Metriken

Stars
 (3.443 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Summary

The game should always operate in landscape mode. However, mobile browsers support screen rotation. Since this results in a viewport changed (and a forced app reload), let's disable screen rotation.

Impacted services

  • The frontend app
  • The game servers
  • Game builds or automation scripts
  • Tests or Github Actions
  • Infrastructure (Terraform)

Steps to reproduce

  1. Open https://staging.duelyst.org in a mobile browser
  2. Rotate the device

Environment information

  • Operating system: Android
  • Browser: Chrome
  • Browser version: Latest

Contributor Guide

Research-Richtung
Die Korrektur beinhaltet das Deaktivieren der Bildschirmrotation auf Mobilgeräten. Wahrscheinlich durch Hinzufügen eines Meta Viewport Tags mit orientation=landscape oder durch Verwendung von CSS zur Sperrung der Ausrichtung. Überprüfen Sie den Frontend Quellcode (index.html oder Hauptlayout). Da keine weiteren Details angegeben sind, bestätigen Sie den Ansatz bei Bedarf mit den Maintainern.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
HTMLCSS
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85