PR

【3/31】育成促進イベント告知表示用 #3425

kouho opened this at 2024-03-27T11:34:57; closed

Description

opened by kouho at 2024-03-27T11:34:57; updated 2024-03-29T17:31:31

概要

育成促進イベントの告知が長くなっていて見づらいため告知をパーツ分けてして整理
本家でもSPに変更のタイミングで報酬お知らせとイベント内容をお知らせで分けていました

  • 変更前

予告

  • 遊び方 + 報酬 + 詳細な情報を表示
  • 変更後

予告

  • 変更点やイベントのポイント
  • イベント内容と報酬の詳細は別お知らせへのリンク

対応内容

  • 予告ヘッダー・イベント進捗・友好度獲得導線リンク・ガチャリンクをパーツ化
  • EventFestivalConfigとPickupGachaConfigへのテンプレートの定義
  • 予告期間中にイベント内容と報酬お知らせを確認可能 & 予告ヘッダーを表示
  • 開催期間のパーツ化

備考

本家では報酬告知のみイベント進捗を掲載していましたが、イベント内容の方にも記載しています。

各告知毎の表示パーツ

| | 予告期間 | 開催中 |
| ------------- | ------------- | ------------- |
| イベント内容 | 予告ヘッダー | イベント進捗<br>友好度獲得導線リンク<br>ガチャリンク |
| 報酬告知l | 予告ヘッダー | イベント進捗<br>友好度獲得導線リンク<br>ガチャリンクl |

確認方法

開発お知らせ
3079, 3080, 3090をインポートして時間旅行で予告期間中とイベント期間中を切り替えて確認してください

Comment

#33857 by kouho at 2024-03-27T12:44:25; updated 2024-03-27T12:44:25

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

Comment

#33866 by wada at 2024-03-27T15:16:41; updated 2024-03-27T15:16:41

@kouho
気になる点まとめました。

予告

・イベント内容や報酬告知から予告に戻れない
→タブが不足している?

報酬

・注意事項が中央揃え

共通

・他のイベント告知に合わせるなら、報酬タブは右側
・期間表示のテキストは「イベント期間」
・下部のライン画像の配置がおかしく、黒背景がはみ出している(div要素の外に出す)
・注意事項「専用ドリンクは…」は、所持アイテムページではなく、専用アイテムページに誘導してほしい(所持アイテムページは見辛い)
・報酬を受け取るボタンがテキスト
・報酬が受け取るボタンを押すと404

Comment

#33867 by wada at 2024-03-27T15:32:06; updated 2024-03-27T15:33:29

@kouho
staicメソッドをあまり使ったことが無いので質問なんですが、staicメソッドを採用する基準は何かありますか?
(PRのconfigクラスメソッドも僕が実装した場合、staticを採用する考えに至らないため)

wada add_label at 2024-03-27T15:32:09: コメントあり

Comment

#33870 by kouho at 2024-03-28T10:01:57; updated 2024-03-28T10:01:57

@wada
確認とご指摘ありがとうございます。

予告に記載している内容は報酬告知とイベント内容にあるため戻るリンクはなくて良いものと考えていました。
>
・イベント内容や報酬告知から予告に戻れない
→タブが不足している?

修正しました
>
報酬
・注意事項が中央揃え

本家に合わせていましたが、他のイベントと合わせてタブの左右修正しました
>
他のイベント告知に合わせるなら、報酬タブは右側

イベント開催をイベント期間に修正しました
>
期間表示のテキストは「イベント期間」

Comment

#33882 by wada at 2024-03-29T11:39:48; updated 2024-03-29T11:39:48

@kouho
修正内容について確認しました。
僕の環境だけかもしれませんが、予告の報酬タブが小さくなっていました。

<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1711679972607D34ySWH4xU" width="400">

Comment

#33883 by kouho at 2024-03-29T12:06:31; updated 2024-03-29T12:06:31

@wada
確認ありがとうございます。確かに予告の報酬タブが小さくなっていましたので開発で調整しました。イベント内容の改行を予告のみ消していみています。

Comment

#33884 by kouho at 2024-03-29T13:36:38; updated 2024-03-29T13:36:38

@wada
以下も修正しました

告知側で修正
> ・注意事項「専用ドリンクは…」は、所持アイテムページではなく、専用アイテムページに誘導してほしい(所持アイテムページは見辛い)

> ソース側で修正
・報酬を受け取るボタンがテキスト
・報酬が受け取るボタンを押すと404

Comment

#33887 by wada at 2024-03-29T14:34:03; updated 2024-03-29T14:34:03

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

wada delete_label at 2024-03-29T14:34:06: コメントあり
wada add_label at 2024-03-29T14:34:08: 和田 LGTM
kouho merge at 2024-03-29T17:31:31: 【3/31】育成促進イベント告知表示用
kouho close at 2024-03-29T17:31:31: Close

Commit range

basedb995e9a9a959903b8fb2edeb00cd3c28f532979
head7b2adac57247880e1c3069b6e192024934a6a16e
base branchjolly/jr-web:event_festival_0401
head branchjolly/jr-web:event_festival_info_parts

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/ajax/default_info.inc
modifiedlib/conf/class/EventFestivalConfig.class.php
modifiedlib/conf/class/PickupGachaConfig.class.php
modifiedpublic_html/template/Main.tpl
Patch body is stored as a separate file to keep this page lightweight.