CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Offen

#78.465 geöffnet am 10.12.2024

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

Repository-Metriken

Stars
 (9.264 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 5h) (292 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
Prüfe Dateien, die 'string input popup.h' enthalten, migriere ein Popup nach dem anderen gemäß dem Muster in #78453 und entferne den Include. Kommentiere, welche Popups schwieriger erscheinen, um Anleitung zu erhalten.
Tech Stack
cpp
Domain
frontend
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.
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++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85