CleverRaven/Cataclysm-DDA

Migrate string popups to ImGui

Open

#78,465 opened on 2024年12月10日

GitHub で見る
 (9 comments) (0 reactions) (0 assignees)C++ (9,264 stars) (3,991 forks)batch import
Good First IssueHelp Wanted[C++]

説明

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

コントリビューターガイド

技術スタック
cpp
領域
desktop
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
C++basic knowledge of ImGui
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85
調査方針
Identify all files including string input popup.h (currently 36). For each file, migrate the popup to use ImGui, following the pattern established in PR #78453. Remove the old popup implementation and its include directive. Start with a single file to minimize risk; if any popup proves difficult, comment in the issue and choose another. Look at the comments in the issue for community guidance.