CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Open

#78.465 geöffnet am 10. Dez. 2024

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (3.991 Forks)batch import
Good First IssueHelp Wanted[C++]

Repository-Metriken

Stars
 (9.264 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 4T 17h) (378 gemergte PRs in 30 T)

Beschreibung

Is your feature request related to a problem? Please describe.

General migration of popups. Usually easy work.

This issue exists to let people know there is an easy contribution they can make if they want.

Ask in this issue or Discord for guidance, if you want.

Solution you would like.

There are currently 36 files with #include "string_input_popup.h" make that 0, then remove the old popup.

  • Migrate them like in #78453.
  • Can be one file at a time per PR (that is usually 1 or 2 popups).
  • If something seems hard, pick another one and comment here which seems harder.

Describe alternatives you have considered.

No response

Additional context

No response

Contributor Guide

Research-Richtung
Identifiziere alle Dateien, die string input popup.h einbinden (derzeit 36). Migriere für jede Datei das Popup zur Verwendung von ImGui, gemäß dem in PR #78453 etablierten Muster. Entferne die alte Popup Implementierung und ihre Include Direktive. Beginne mit einer einzelnen Datei, um das Risiko zu minimieren; falls ein Popup schwierig ist, kommentiere im Issue und wähle ein anderes. Siehe die Kommentare im Issue für Anleitungen der Community.
Tech Stack
cpp
Domain
desktop
Issue Type
Refactoring
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++basic knowledge of ImGui
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85