PR

【2/18】天下無双イベント #606

iwamoto opened this at 2021-02-10T19:54:58; closed

Description

opened by iwamoto at 2021-02-10T19:54:58; updated 2023-11-27T13:40:28

【2/18】天下無双イベント #599

*今回は船ボーナスがループではない

iwamoto change_title at 2021-02-10T20:25:02: 【2/18】天下無双イベント [WIP]【2/18】天下無双イベント
iwamoto change_title at 2021-02-15T09:46:02: [WIP]【2/18】天下無双イベント 【2/18】天下無双イベント

Comment

#9414 by iwamoto at 2021-02-15T09:47:08; updated 2021-02-15T09:47:08

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

iwamoto add_label at 2021-02-15T11:29:03: レビュー待ち

Comment

#9418 by wada at 2021-02-15T14:29:18; updated 2021-02-15T14:29:18

@iwamoto
subfunc.incに設定が残っているのはよろしくないので、船ボーナスの設定をconfigファイルに移して欲しいです。
日数も見辛いので計算式で算出してもらって良いです。

int day_count = roundup((time - start) / 86400);

Comment

#9420 by iwamoto at 2021-02-16T10:04:37; updated 2021-02-16T10:04:37

@wada
指摘箇所修正対応を行いました。
ご確認お願いします。

Comment

#9423 by iwamoto at 2021-02-16T10:21:50; updated 2021-02-16T10:21:50

@wada
見落としていました、修正対応を行いました。

Comment

#9424 by wada at 2021-02-16T10:40:21; updated 2021-02-16T10:40:21

手配書の残り時間表記を変更して欲しいです。
分と秒をとるとアトの意味が伝わりにくくなるので、残りに変更を。
アトo分o秒残り00:00

!スクリーンショット 2021-02-16 10

Comment

#9425 by wada at 2021-02-16T10:50:28; updated 2021-02-16T10:50:28

メインページのイベントダイアログで、ランキング報酬が表示されますが、
700位以上は全てカード名表記で代わり映えしないので、冒険と同じ「限定カードx10」にしたいです。

!スクリーンショット 2021-02-16 10

Comment

#9426 by iwamoto at 2021-02-16T10:53:40; updated 2021-02-16T10:53:40

@wada
承知しました。
他のイベントのときも同様の表示になっていますのでこの際に統一します

ただ、下記のように残りが時間単位である場合はどのようにしましょうか?

$("#page_mission_default_countdown2_span").html("アト"+hour+"時間"+min+"分");

Comment

#9428 by wada at 2021-02-16T11:17:32; updated 2021-02-16T11:17:32

勘違いしてた、単位の話ね。

表示内容によって変わるけど、手配書モードは、
・15分以上にならない
・jsで1秒ごとに更新される
ことから、残り 00:00表記が良いと思っています。
(必勝フラッグの表示と合わせたいというのも)

1時間以上超えた時は、その機能や表示箇所によって表示方法を変えるべきかなと。

Comment

#9429 by wada at 2021-02-16T12:04:22; updated 2021-02-16T12:04:22

@iwamoto
レビューOKです。
残り時間と報酬名対応お願いします。

Comment

#9434 by iwamoto at 2021-02-16T12:13:21; updated 2021-02-16T12:13:21

@wada
ありがとうございます。

また、残り時間の表記 & メインページのイベントダイアログにて表示される報酬名を変更しました

Comment

#9435 by wada at 2021-02-16T12:34:00; updated 2021-02-16T12:34:00

@iwamoto
確認しました。OKです。

iwamoto merge at 2021-02-16T13:48:19: 【2/18】天下無双イベント
iwamoto close at 2021-02-16T13:48:19: Close

Commit range

basecf9dce6cf640091883b9d5a04364e7fa4d26993c
head44d1ea8dc5c878fd10fd4c15ee5740bc0496be52
base branchjolly/jr-web:develop
head branchjolly/jr-web:event_battle_7_599

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed10 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/conf/class/BattleEventConfig.class.php
modifiedlib/conf/define_y.inc
modifiedlib/exec/tool_event_battle_present.inc
modifiedlib/public/Main.inc
addedpublic_html/img/ranking/event_battle/info7.jpg
modifiedpublic_html/js/JOLLYROGER_mission.js
modifiedpublic_html/js/jr_mission.js
modifiedpublic_html/template/Main.tpl
modifiedpublic_html/template/Mission.tpl
Patch body is stored as a separate file to keep this page lightweight.