PR

【1/28】冒険変速対応 #1508

wada opened this at 2022-01-26T17:53:19; closed

Description

opened by wada at 2022-01-26T17:53:19; updated 2022-02-07T12:19:32

概要

【2/7】冒険変速対応 #1504
冒険の演出スピードを3段階で設定、保存

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

可能であれば、端末でも確認お願いします。

Comment

#16886 by wada at 2022-01-26T17:53:32; updated 2022-01-26T17:53:32

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

wada add_label at 2022-01-26T17:53:35: レビュー待ち

Comment

#16900 by iwamoto at 2022-01-27T11:58:22; updated 2022-01-27T11:58:22

@wada
確認しました。動作には問題ないように思います。
ただ、イベントでの画面切り替え時にspeed_upボタンだけ表示されてしまっています。

!newisland
!raid

Comment

#16903 by kouho at 2022-01-27T15:31:57; updated 2022-01-27T15:31:57

@wada
岩本さんも指摘してくれていますが画面切り替え時にspeed_upボタンが残っている点以外は問題ないよう見えます。

Comment

#16904 by wada at 2022-01-27T16:30:42; updated 2022-01-27T16:30:42

@iwamoto @kouho
演出中はspeed_upない方が良いですか?

Comment

#16905 by iwamoto at 2022-01-27T16:36:11; updated 2022-01-27T16:36:11

@wada
いえ、演出中も速度変更できるようにと意図してあるなら大丈夫かと思います。
見栄えだけだと、ボタンが浮いていると感じましたのであげました。

Comment

#16908 by kouho at 2022-01-27T16:55:39; updated 2022-01-27T16:55:39

@wada
下の画像のような背景が消えますので、
演出中はボタンがない方が違和感がないように見えましたが、本家でも同様であれば大丈夫です。
!スクリーンショット 2022-01-27 16

Comment

#16909 by wada at 2022-01-27T17:54:38; updated 2022-01-27T17:54:55

@iwamoto @kouho
了解です。
表示の違和感(ボタンが浮く)とプレイの不自由感(演出中にspped_upできない)を比較した時に、不自由感を排除した方が良いと判断したため、ボタン画像は演出中も表示します。

画像の変更で軽減できそうなら対処します。

Comment

#16910 by kouho at 2022-01-27T18:14:39; updated 2022-01-27T18:14:39

@wada
かしこまりました。

kouho add_label at 2022-01-27T18:14:43: 皇甫 LGTM

Comment

#16912 by iwamoto at 2022-01-27T18:14:52; updated 2022-01-27T18:14:52

@wada
はい、承知しました。
そちらで問題ないと思います。

iwamoto add_label at 2022-01-27T18:14:56: 岩本 LGTM
wada merge at 2022-01-31T10:17:35: 【1/28】冒険変速対応
wada close at 2022-01-31T10:17:35: Close
wada refer at 2022-01-31T11:07:35: 1519:【1/31】冒険変速告知画像
wada refer at 2022-02-07T12:19:32: 1536:【2/7 14:00】冒険変速リリース

Commit range

basea880b225b4a7d907011d7a6ea280a4de42f2fa84
head64cd166a9ec72a46abcc190589b0f3b7abb9c513
base branchjolly/jr-web:develop
head branchjolly/jr-web:mission_speed

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed12 filesView patch
statusfile
addedlib/ajax/mission_save_speed.inc
modifiedlib/public/Mission.inc
modifiedpublic_html/ajax.php
addedpublic_html/img/mission/return_0.png
addedpublic_html/img/mission/return_1.png
addedpublic_html/img/mission/speedup_1.png
addedpublic_html/img/mission/speedup_2.png
addedpublic_html/img/mission/speedup_3.png
modifiedpublic_html/js/JOLLYROGER_mission.js
modifiedpublic_html/template/Card-ExpItem.tpl
modifiedpublic_html/template/Jewel.tpl
modifiedpublic_html/template/Mission.tpl
Patch body is stored as a separate file to keep this page lightweight.