diasurgical/DevilutionX

[Issue Report]: Full screen option on Nintendo Switch

Offen

#7.032 geöffnet am 17.03.2024

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (743 Forks)batch import
good first issue

Repository-Metriken

Stars
 (7.255 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Operating System

Nintendo Switch

DevilutionX version

1.5.2

Describe

I guess the full screen option is meant for desktop versions of the app. On consoles like Nintendo Switch it has no sense, as there's not a desktop. So, enabling it, creates some weird results, depending on the set screen dimension (480p, 720p, 1080p):

SI_1

SI_2

Maybe is it better to remove it in the next releases?

I also noticed other options that seem not useful at all on a console, like the keyboard shortcuts settings (the gamepad one is the only working here).

To Reproduce

Settings > Graphics > Full screen

Expected Behavior

No option at all.

Additional context

No response

Contributor Guide

Research-Richtung
Untersuchen Sie den Code des Einstellungsmenüs, um die Vollbildoption auf Konsolenplattformen wie Nintendo Switch bedingt auszublenden. Stellen Sie sicher, dass die Option nur auf Desktop Plattformen angezeigt wird.
Tech Stack
cpp
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70