Skip to content

海域名と連戦数を表示する機能#1779

Open
developerfred wants to merge 1 commit intoKanCraft:developfrom
developerfred:develop
Open

海域名と連戦数を表示する機能#1779
developerfred wants to merge 1 commit intoKanCraft:developfrom
developerfred:develop

Conversation

@developerfred
Copy link
Copy Markdown

概要

issue #1764 で要望されていた「海域名と連戦数を表示する機能」を実装しました。

変更内容

Message.ts

  • 大破進撃防止窓のキャプチャ時に Logbook.sortie から海域情報と連戦数を取得
  • 別窓表示モード時に海域情報(seaArea)と連戦数(battleCount)をメッセージに追加

DamageSnapshotPage.tsx

  • 受信した海域情報と連戦数を使用してタイトルを動的に更新
  • 形式: {海域} ({連戦数}) - 艦隊状況 (例: 1-1 (3) - 艦隊状況)

動作確認

  • ビルドが成功することを確認
  • テストがパスすることを確認
  • 既存の機能に影響がないことを確認

関連issue

Closes #1764

補足

v3で存在していた機能をv4に移植しました。v3では大破進撃窓に 1-1 (1) のように表示されていた機能を再現しています。

- Message.ts: 大破進撃防止窓のキャプチャ時に海域情報と連戦数を取得して送信
- DamageSnapshotPage.tsx: タイトルを海域と連戦数で更新する機能を実装
- v3で存在していた機能をv4に移植完了
@otiai10 otiai10 self-requested a review January 12, 2026 02:24
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 12, 2026

Codecov Report

❌ Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 3.36%. Comparing base (ef14aa0) to head (86f4333).

Files with missing lines Patch % Lines
src/controllers/Message.ts 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #1779      +/-   ##
==========================================
- Coverage     3.37%   3.36%   -0.01%     
==========================================
  Files           36      36              
  Lines         1361    1365       +4     
  Branches        46      46              
==========================================
  Hits            46      46              
- Misses        1315    1319       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@otiai10 otiai10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。そもそも、 Logbook.sortie に格納されている .map の情報が、正しいものが正しいタイミングで取得できているか、というところにもバグがある可能性があります。この点、どう考えられていますか?

また、ご自身は艦これウィジェットのどのバージョンを現在使われていますか?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

海域名と連戦数を表示する機能

2 participants