PR

【11/4 or 5】新大陸開催時看板に勝利の証所持数追加 #1287

kouho opened this at 2021-11-04T17:59:31; closed

Description

opened by kouho at 2021-11-04T17:59:31; updated 2026-01-05T14:09:16

概要

新大陸開催時に勝利の証所持数を看板位追加するPRです。
10/11 のエンジニアMTGで話していた分の対応です。

フックモンスター討伐の報酬は勝利の証で決め打ちしています。

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

  • レイアウトを調整しましたので、レイアウトがおかしくないか
  • 勝利の証の画像と所持数は正しく表示されるか

TODO

特になし

kouho assign at 2021-11-04T17:59:34: Not assigned:kouho
kouho add_label at 2021-11-04T17:59:51: レビュー待ち
kouho add_label at 2021-11-04T17:59:55: イベント
kouho add_label at 2021-11-04T17:59:57: デベロップへマージ

Comment

#14983 by kouho at 2021-11-04T18:00:37; updated 2021-11-04T18:00:37

@wada @iwamoto
確認お願いいたします。

Comment

#14989 by wada at 2021-11-04T18:38:15; updated 2021-11-04T18:38:15

@kouho
仲介所へのリンク画像は今回追加しないで良いですかね。
https://docs.google.com/spreadsheets/d/1hMIqcSRE4mWfZhlJYPVYC7bjQ1IcrsWiXvtrwohXadc/edit#gid=1551633191

Comment

#14990 by kouho at 2021-11-04T18:45:47; updated 2021-11-04T18:45:47

@wada
すみません、仕様書の記載見落としていました。
仲介所へのリンク画像は追加します。レイアウトも若干違いますので修正します。。。

Comment

#14992 by wada at 2021-11-04T18:49:45; updated 2021-11-04T18:49:45

@kouho
今の表示でも問題ないので、時間があれば修正お願いします。

Comment

#14993 by kouho at 2021-11-04T19:19:45; updated 2021-11-04T19:23:37

@wada
了解です。若干違いますが、仕様書に合わせて表示を少し修正しました。
仲介所へのリンクは一旦勝利の証画像から貼っています。

ご指摘いただいた#page_main_default_event_info_modal_window_body_card_piece_name_newislandには勝利の証のテキストを割り当てています。

Comment

#15030 by wada at 2021-11-05T18:16:07; updated 2021-11-05T18:16:07

@kouho
ありがとうございます。表示内容は問題ありません。
「勝利の証」のフォントが見づらいので、boldかshadowをかけてもらえると。

wada add_label at 2021-11-05T18:16:14: 和田 LGTM

Comment

#15035 by iwamoto at 2021-11-05T19:05:02; updated 2021-11-05T19:05:02

@kouho
確認しました。問題ないと思います。

iwamoto add_label at 2021-11-05T19:05:05: 岩本 LGTM

Comment

#15088 by kouho at 2021-11-09T11:01:21; updated 2021-11-09T11:01:21

@wada @iwamoto
こちら仲介所へのリンク画像も追加しましたので再度ご確認いただけないでしょうか?

Comment

#15090 by wada at 2021-11-09T11:26:54; updated 2021-11-09T11:26:54

@kouho
画像ファイルがないみたいです。

Comment

#15091 by kouho at 2021-11-09T11:42:13; updated 2021-11-09T11:42:13

@wada
すみません、画像ファイルコミットしました

Comment

#15099 by wada at 2021-11-09T13:44:00; updated 2021-11-09T13:44:00

@kouho
ありがとうございます。
ボタンを追加したので、ユーザーの意図しない箇所がリンクにならないよう、勝利の証画像のリンクを外して欲しいです。
僕の環境だけかもしれませんが、報酬のボタンが上にずれているので、確認をおねがいします。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1636432828656xLC7gsnd07" width="300">

Comment

#15100 by kouho at 2021-11-09T13:58:31; updated 2021-11-09T13:58:31

@wada
確認ありがとうございます。以下対応行いました。

  • 報酬のボタンずれの修正
  • 勝利の証画像リンク削除
  • 勝利の証テキストにboldとshadowを追加。(どちらかという話でしたがより見やすくするためどちらもつけてみました。)

Comment

#15101 by kouho at 2021-11-09T14:01:43; updated 2021-11-09T14:01:43

↑の表示修正コミットできていませんでしたので改めてコミットしました。

Comment

#15104 by wada at 2021-11-09T14:40:35; updated 2021-11-09T14:40:35

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

Comment

#15105 by iwamoto at 2021-11-09T16:08:21; updated 2021-11-09T16:08:21

@kouho
修正確認しました。
問題ないと思います。

kouho merge at 2021-11-09T16:13:24: 【11/4 or 5】新大陸開催時看板に勝利の証所持数追加
kouho close at 2021-11-09T16:13:24: Close
kouho refer at 2021-11-11T10:28:13: 1300:【11/11】リリース

Commit range

base3f6aa05ddc4008d3ff695facc765aa04112a8f5d
head89e6fe618a4b9a3aa5b19f038f2b28e66385eed5
base branchjolly/jr-web:develop
head branchjolly/jr-web:board_piece

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed4 filesView patch
statusfile
modifiedlib/public/Main.inc
addedpublic_html/img/main/event_info_window/newisland/status_exchange.png
modifiedpublic_html/js/JOLLYROGER.js
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.