PR

交易結果画面の覚醒合成導線改善 #3452

kouho opened this at 2024-04-11T10:00:06; closed

Description

opened by kouho at 2024-04-11T10:00:06; updated 2024-04-24T18:46:59

概要

徳永さん要望のtrelloカードへの対応

https://trello.com/c/pFvQve2f/503-%E4%BA%A4%E6%98%93%E8%88%B9%E3%81%AE%E4%B9%97%E7%B5%84%E5%93%A1%E8%A9%B3%E7%B4%B0%E3%83%9A%E3%83%BC%E3%82%B8%E3%81%AB%E8%A6%9A%E9%86%92%E5%90%88%E6%88%90%E3%83%9C%E3%82%BF%E3%83%B3%E8%A8%AD%E7%BD%AE

交易結果画面への改善対応

[元々]

  • 表示条件

最終覚醒をしていないレベルMAXのカードが交易から返ってきた場合

  • 表示箇所

「カードを変更する」のボタンに横に覚醒合成ボタンを表示

  • 覚醒合成ボタンからのリンク

覚醒合成のTOPページ

[改善後]

  • 表示条件

最終覚醒をしていないレベルMAXのカードが交易から返ってきた場合

  • 表示箇所

最終覚醒をしていないレベルMAXのカードのステータスの下に覚醒合成ボタンをカードごとに設置

  • 覚醒合成ボタンからのリンク

カードを覚醒者に選択した状態

kouho assign at 2024-04-11T10:00:20: Not assigned:kouho
kouho add_label at 2024-04-11T10:00:23: マスタへマージ
kouho add_label at 2024-04-11T10:00:24: レビュー待ち
kouho change_title at 2024-04-11T10:12:47: 【】交易画面に覚醒合成導線 交易画面に覚醒合成導線

Comment

#34127 by kouho at 2024-04-11T10:23:33; updated 2024-04-11T10:23:33

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

kouho change_title at 2024-04-11T10:33:50: 交易画面に覚醒合成導線 交易結果画面の覚醒合成導線改善

Comment

#34211 by wada at 2024-04-17T13:21:25; updated 2024-04-17T13:21:25

@kouho
僕が聞いていた内容だと、乗組員の変更ページ(/?M=Commerce&A=Member)に追加してほしいとのことでした。
こちらが正しいということですかね。それとも追加で対応してほしいとのことでしたか?

Comment

#34212 by kouho at 2024-04-17T13:55:58; updated 2024-04-17T13:55:58

@wada
若干聞いている話が異なりますね…直接月曜日にでも会ったときに確認しますか?

Comment

#34251 by wada at 2024-04-19T16:18:39; updated 2024-04-19T16:18:39

@kouho
これは僕の感覚ですが、カード情報を表示仕切ってからボタンを配置したほうが良さそうな感じがあります。
画像だとスキル情報の下です。
<img src="https://git.dev.d-es.co.jp/gitbucket/jolly/jr-web/_attached/1713510996775ybPwBqnBqb" width="400">

wada add_label at 2024-04-19T16:20:11: 和田 LGTM

Comment

#34253 by kouho at 2024-04-19T16:25:05; updated 2024-04-19T16:25:05

@wada
確かに中途半端な位置にありますね。修正します。

Comment

#34254 by kouho at 2024-04-19T16:35:39; updated 2024-04-19T16:38:34

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

> $_temp['card_max_lv_flg'] の用途が無くなったので削除して良いと思います。

$_temp['card_max_lv_flg'] フラグを削除してif文に埋め込みました

> これは僕の感覚ですが、カード情報を表示仕切ってからボタンを配置したほうが良さそうな感じがあります。
画像だとスキル情報の下です。

覚醒合成ボタンをスキル説明の下に配置しました

Comment

#34255 by wada at 2024-04-19T16:57:18; updated 2024-04-19T16:57:18

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

kouho merge at 2024-04-24T18:46:59: 交易結果画面の覚醒合成導線改善
kouho close at 2024-04-24T18:46:59: Close

commit comment

#224 by wada at 2024-04-19T16:10:19; public_html/template/Commerce-Result.tpl / old:None new:49
7f7f43d10ac538ccadad87cde409d84457c2ed13

@kouho
$_temp['card_max_lv_flg'] の用途が無くなったので削除して良いと思います。

Commit range

base1f6c3f19695a4e6d332e4b8f4746d76ce4063056
head7f7f43d10ac538ccadad87cde409d84457c2ed13
base branchjolly/jr-web:master
head branchjolly/jr-web:commerce_composition

View commits

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

Review comment

#224 by wada at 2024-04-19T16:10:19; public_html/template/Commerce-Result.tpl / old:None new:49
7f7f43d10ac538ccadad87cde409d84457c2ed13

@kouho
$_temp['card_max_lv_flg'] の用途が無くなったので削除して良いと思います。