CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Open

#78.465 aperta il 10 dic 2024

Vedi su GitHub
 (9 commenti) (0 reazioni) (0 assegnatari)C++ (3991 fork)batch import
Good First IssueHelp Wanted[C++]

Metriche repository

Star
 (9264 star)
Metriche merge PR
 (Merge medio 4g 17h) (378 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Identifica tutti i file che includono string input popup.h (attualmente 36). Per ciascun file, migra il popup per utilizzare ImGui, seguendo lo schema stabilito nella PR #78453. Rimuovi la vecchia implementazione del popup e la sua direttiva di inclusione. Inizia con un singolo file per minimizzare il rischio; se un popup risulta difficile, commenta nell'issue e scegline un altro. Guarda i commenti nell'issue per le indicazioni della comunità.
Tech stack
cpp
Dominio
desktop
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++basic knowledge of ImGui
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85