CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Ouverte

#78 465 ouverte le 10 déc. 2024

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée)C++ (3 991 forks)batch import
Good First IssueHelp Wanted[C++]

Métriques du dépôt

Stars
 (9 264 étoiles)
Métriques de merge PR
 (Merge moyen 6j 5h) (292 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Inspectez les fichiers qui incluent 'string input popup.h', migrez une fenêtre contextuelle à la fois en suivant le modèle dans #78453, et supprimez l'inclusion. Commentez les fenêtres qui semblent plus difficiles pour obtenir des conseils.
Stack technique
cpp
Domaine
frontend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85