CleverRaven / CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Open
#78,465 9 comments 0 reactions 0 assignees View on GitHub
[C++] Good First Issue Help Wanted
Dominant language
C++
Stars
13.1k
Forks
4.6k
Avg merge
2d 1h
Merged PRs (30d)
200

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_

Contributor guide

Open the contributing guide

Research direction

Search the 36 files that include string_input_popup.h and use issue #78453 as the migration example. Start with one file and confirm how its one or two popups are migrated; done means the selected file no longer includes the old header, with the old popup removable once all files are migrated.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.