Archive / jolly / jr-web
PR

バトルリザルトスキル詳細確認可能に(本家移植) #2375

kouho opened this at 2022-11-02T15:04:18; closed

Description

opened by kouho at 2022-11-02T15:04:18; updated 2022-12-02T10:57:24

概要

以下の本家対応移植用PRです
バトルリザルトでスキル詳細を見れるように
https://git.dev.d-es.co.jp/gitbucket/jolly/jr-cs-issue/issues/180

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

バトル結果画面でスキルをタップするとモーダルが表示され、スキル詳細を確認できる

TODO

確認出来次第開発に反映しデバッグ

本家対応

https://github.com/rzstaffgit/jrce2_github/pull/8512

Comment

#23402 by kouho at 2022-11-02T15:05:39; updated 2022-11-02T15:05:39

@wada
developをマージ先にして本家の対応元PRリンクを貼りましたので改めて確認よろしくお願いいたします

Comment

#23407 by wada at 2022-11-02T15:19:24; updated 2022-11-02T15:19:24

@kouho
移植元のPRと更新内容が一致しないので、
直前のバトル結果でスキル詳細が表示されるかだけ確認で良いですか？

Comment

#23408 by kouho at 2022-11-02T15:23:50; updated 2022-11-02T15:23:50

@wada
その確認で大丈夫です。

Comment

#23409 by wada at 2022-11-02T16:44:02; updated 2022-11-02T16:44:11

@kouho
閉じるボタンがテキストに被っているので、少しずらして欲しいです。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1667374997881TGxHvB9fVw" width="300">

Comment

#23410 by kouho at 2022-11-02T17:07:12; updated 2022-11-02T17:07:12

@wada
確認ありがとうございます。
> $_temp['archive_id']が存在しない変数です。
> log出力が残っています。
> 使用しないのであれば削除して欲しいです。
> ここのflgはskill_open_flgでなくて良いですか？
> 閉じるボタンがテキストに被っているので、少しずらして欲しいです。

指摘していただいた箇所修正いたしました。

Comment

#23626 by wada at 2022-11-11T14:26:41; updated 2022-11-11T14:26:41

@kouho
ありがとうございます。確認OKです。

wada add_label at 2022-11-11T14:26:45: 和田 LGTM
kouho merge at 2022-11-18T11:09:17: バトルリザルトスキル詳細確認可能に(本家移植)
kouho close at 2022-11-18T11:09:17: Close
kouho refer at 2022-12-02T10:57:24: 2444:【12/2】本家スキルマージ

commit comment

#143 by wada at 2022-11-02T16:08:39; lib/public/Battle-Result.inc / old:None new:30
a94f6fb36440d76cc1d917fe37f10de06caccfef

@kouho
$_temp['archive_id']が存在しない変数です。

commit comment

#144 by wada at 2022-11-02T16:10:39; public_html/js/JOLLYROGER_battle.js / old:None new:20854
45bee33946cb943868ed957bae7b19d4929da1d4

@kouho
log出力が残っています。

commit comment

#145 by wada at 2022-11-02T16:11:39; public_html/js/JOLLYROGER_battle.js / old:None new:20874
45bee33946cb943868ed957bae7b19d4929da1d4

@kouho
使用しないのであれば削除して欲しいです。

commit comment

#146 by wada at 2022-11-02T16:21:39; public_html/js/JOLLYROGER_battle.js / old:None new:20870
a94f6fb36440d76cc1d917fe37f10de06caccfef

@kouho
ここのflgはskill_open_flgでなくて良いですか？

Commit range

basefa25b97a714b9d737e54ba980ea31944ac3b319a
head45bee33946cb943868ed957bae7b19d4929da1d4
base branchjolly/jr-web:develop
head branchjolly/jr-web:battle_result_skill

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed3 filesView patch
statusfile
modifiedlib/public/Battle-Result.inc
modifiedpublic_html/js/JOLLYROGER_battle.js
modifiedpublic_html/template/Battle-Result.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#143 by wada at 2022-11-02T16:08:39; lib/public/Battle-Result.inc / old:None new:30
a94f6fb36440d76cc1d917fe37f10de06caccfef

@kouho
$_temp['archive_id']が存在しない変数です。

Review comment

#144 by wada at 2022-11-02T16:10:39; public_html/js/JOLLYROGER_battle.js / old:None new:20854
45bee33946cb943868ed957bae7b19d4929da1d4

@kouho
log出力が残っています。

Review comment

#145 by wada at 2022-11-02T16:11:39; public_html/js/JOLLYROGER_battle.js / old:None new:20874
45bee33946cb943868ed957bae7b19d4929da1d4

@kouho
使用しないのであれば削除して欲しいです。

Review comment

#146 by wada at 2022-11-02T16:21:39; public_html/js/JOLLYROGER_battle.js / old:None new:20870
a94f6fb36440d76cc1d917fe37f10de06caccfef

@kouho
ここのflgはskill_open_flgでなくて良いですか？