PR

ラインナップリセット表示にdisplay noneを入れておく #1646

kouho opened this at 2022-03-17T19:33:58; closed

Description

opened by kouho at 2022-03-17T19:33:58; updated 2022-03-18T14:48:57

概要

通信環境や端末の処理速度が非常に遅い場合にhtmlは読み込んだがjsの.hide()まで読めていない状態だとリセットできる表記が出る可能性があることへの対応。

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

リセットできる表示が問題なく表示されていることの確認

TODO

Comment

#17924 by wada at 2022-03-18T10:14:12; updated 2022-03-18T10:14:12

@kouho
ありがとうございます。他のresetableの箇所も非表示で大丈夫です。

<div class="resetable fontbold" style="text-align:center;color:yellow;">ラインナップをリセットできます</div>

Comment

#17930 by kouho at 2022-03-18T11:53:27; updated 2022-03-18T11:53:27

@wada
確認ありがとうございます。他のresetableの箇所も非表示にしました。

Comment

#17931 by kouho at 2022-03-18T13:18:49; updated 2022-03-18T13:18:49

@wada
すみません、修正しました。。。

Comment

#17932 by wada at 2022-03-18T14:18:13; updated 2022-03-18T14:18:13

@kouho
ありがとうございます。
EventGacha-Default.tpl のここだけ対応が反映されていないみたいです。

<div class="resetable fontbold" style="text-align:center;color:yellow;display none;">ラインナップをリセットできます</div>

Comment

#17934 by kouho at 2022-03-18T14:23:32; updated 2022-03-18T14:23:32

@wada
確認ありがとうございます。確かに反映できていませんでしたのでEventGacha-Default.tpl修正しました。

Comment

#17935 by wada at 2022-03-18T14:44:09; updated 2022-03-18T14:44:09

@kouho
確認しました。okです

wada add_label at 2022-03-18T14:44:12: 和田 LGTM
kouho merge at 2022-03-18T14:48:57: ラインナップリセット表示にdisplay noneを入れておく
kouho close at 2022-03-18T14:48:57: Close

commit comment

#114 by wada at 2022-03-18T12:23:58; public_html/template/EventGacha-Default.tpl / old:None new:98
94416ddb8c864b28bb3aa3519170588066c818fd

@kouho
すいません、displaynone:で区切って欲しいです。

Commit range

base07f4a702a6394b9d931391e021a0c5d78b0fa16b
head23c6f8fd911fa380c9a02318cb3f6938fcc7fa7e
base branchjolly/jr-web:develop
head branchjolly/jr-web:fix_event_gacha_reset

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed2 filesView patch
statusfile
modifiedpublic_html/template/EventGacha-Default.tpl
modifiedpublic_html/template/EventGacha-GachaResult.tpl
Patch body is stored as a separate file to keep this page lightweight.

Review comment

#114 by wada at 2022-03-18T12:23:58; public_html/template/EventGacha-Default.tpl / old:None new:98
94416ddb8c864b28bb3aa3519170588066c818fd

@kouho
すいません、displaynone:で区切って欲しいです。