CleverRaven / CleverRaven/Cataclysm-DDA
Migrate string popups to ImGui
- 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
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