PR

スキル複数時の表示修正 #2164

wada opened this at 2022-08-03T14:36:14; closed

Description

opened by wada at 2022-08-03T14:36:14; updated 2022-08-08T15:22:12

概要

船スキルが複数になるとスキル詳細の表示がおかしくなる #2145
造船リストの船詳細で、スキルとエンブレム能力を2つ持つ場合、スキルの表示が混ざってしまう

修正前

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1659504728881wBqQVXRkCj" width="300">
凶撃がエンブレム名で上書きされている

修正後

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1659504733033AqDBPJRsZ4" width="300">

対象船

  • 10002: シロナガスン lv26

ペースアップ、交易の船の速度が2倍

  • 10003:キャラック lv26

凶撃、デスペナルティ1分短縮

テスト方法、確認した内容

キャラック、シロナガスンをlv26まで解放して、造船リストから船詳細のスキル詳細を確認

Comment

#21345 by wada at 2022-08-03T14:36:26; updated 2022-08-03T14:36:26

@kouho
レビューお願いします。

Comment

#21360 by kouho at 2022-08-03T17:58:15; updated 2022-08-03T17:58:15

@wada
Ship-Detailで最大値ボタンを押さない場合の表示も修正していただきたいです。
実際に船のレベルがスキルとエンブレム能力を2つ持てるレベルになっている場合にエンブレム効果しか表示されませんでした。

Comment

#21371 by wada at 2022-08-03T19:26:31; updated 2022-08-03T19:26:31

@kouho
確認ありがとうございます。
こちらでlv26まで強化したところ、どちらも表示されました。

確認した手順としては、以下の通りです。
1. master_cardのmax_lvを26に変更
2. master_ship_dataのlv22~26を解放
3. 対象の船を作成
4. user_card_holderのlvを25まで上げる
5. 船強化でlvを26に上げる

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1659522072470MtbACRv5IE" width="300">

kouho add_label at 2022-08-04T09:57:53: 皇甫 LGTM

Comment

#21374 by kouho at 2022-08-04T09:58:05; updated 2022-08-04T09:58:05

@wada
確認OKです

wada merge at 2022-08-04T11:35:10: スキル複数時の表示修正
wada close at 2022-08-04T11:35:10: Close
wada refer at 2022-08-04T13:37:49: 2166:[WIP]【8/8】更新リリース
wada refer at 2022-08-08T10:06:08: 2166:【8/8】更新リリース
wada delete_branch at 2022-08-08T15:22:12: fix_ship_multi_skill_2145

Commit range

basefca71637eaf774a75e0f655616cc9fb174380af4
head3ae532373a5e1cd31f1b4440d1c0684b050a4f5e
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_ship_multi_skill_2145

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed1 filesView patch
statusfile
modifiedlib/public/Ship-SelectDetail.inc
Patch body is stored as a separate file to keep this page lightweight.