[フォトアルバム]動画の再生を一覧画面か詳細画面を選べるようにしました。#2308
Conversation
iPhoneの使用時に、通信環境の良くないところでは、動画が多数あると再生できずに、数が少ないと再生できる現象を経験しています。 そのため、一覧画面で再生できるパターンと詳細画面でひとつずつ再生できるパターンを設定できるようにしました。
|
表示設定の画面です。 |
|
詳細画面を使わない場合。 |
|
「一覧はサムネイル画像のみで詳細画面で再生する」パターンの一覧での動画の画面。 |
|
「一覧はサムネイル画像のみで詳細画面で再生する」の場合の動画の詳細画面 |
gakigaki
left a comment
gakigaki
left a comment
There was a problem hiding this comment.
ご対応ありがとうございます。
動画がたくさんあるフォトアルバムだと、とても有効な表示形式だと思います。
コメントを参照の上、ご対応をお願いします。
| */ | ||
| public function detail($request, $page_id, $frame_id, $photoalbum_content_id) | ||
| { | ||
| // 対象のデータを取得して詳細画面を表示する。 |
There was a problem hiding this comment.
detail() で PhotoalbumContent::find() 結果の存在チェックやバケツ一致確認がありません。
存在しない/他アルバムのIDを指定すると 500 になるか、別アルバムの動画がこのフレームで再生・パンくず表示されます。
index() と同様にバケツの確認をし、さらにコンテンツの存在確認、見つからなければ 404リダイレクトする必要があります。
| /** | ||
| * 表示する・表示しない区分 | ||
| */ | ||
| final class PhotoalbumPlayviewType extends EnumsBase |
There was a problem hiding this comment.
このクラスがどこからも参照されておらず、ラジオの値はリテラル値 0/1 になっています。
未使用なら削除するか、ラジオのラベルと値で利用するかして、混乱を避けたいです。
| <label class="custom-control-label" | ||
| for="{{ "play_view_1" }}" | ||
| id="{{ "label_play_view_1" }}"> | ||
| 一覧はサムネイル画像のみで詳細画面で再生する |
There was a problem hiding this comment.
「一覧」の対になるので、「詳細画面で再生する」で十分伝わると思います。
|
|
||
| {{-- 詳細画面がある場合の一覧での説明表示文字数 --}} | ||
| <div class="form-group row"> | ||
| <label class="{{$frame->getSettingLabelClass(true)}}">{{PhotoalbumFrameConfig::enum[PhotoalbumFrameConfig::description_list_length]}}</label> |
There was a problem hiding this comment.
画像のようにすると、より項目の意味合いが伝わりやすくなると思います。
また、「詳細画面で再生する」の場合のみ、文字数の項目を活性化するとより親切なUIになるかと思います。
2 participants
概要
iPhoneの使用時に、通信環境の良くないところでは、動画が多数あると再生できずに、数が少ないと再生できる現象を経験しています。 そのため、一覧画面で再生できるパターンと詳細画面でひとつずつ再生できるパターンを設定できるようにしました。
レビュー完了希望日
関連Pull requests/Issues
参考
DB変更の有無
無し
チェックリスト