-
Notifications
You must be signed in to change notification settings - Fork 1
Feature/create history viewer2 #135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@Makochaaan お疲れ様、jackFesまでにレビュー間に合わなくてゴメンナサイ... モバイル版のスタイリング2025-12-20.17.01.37.1.mp4レスポンシブ対応は多分まだ取り組めていないところだろうけど、モバイルレイアウトの際に、ベースのテキストのみが表示される不具合を発見しました。 おそらく、 UX関連2025-12-20.17.23.57.mp42段目の動画クリックに関してPCで履歴画面を操作する際に、上から2段目以降の動画のクリックに関していくつか感じたことがあるので述べます。
改善案として、動画のテキスト表示を
が考えられると思いました。 |
|
ここのスコープ外ですが、 |
issue番号(- #〇)
やったこと(動作の確認のため、ある場合は動画や画像を添付してください)
#97 を引き継ぎ、実装を完了させました。
完成イメージ:
<video src=""/>
[とくに見て欲しいところ]
。Discordで共有した通りです。
https://discord.com/channels/1420354588388753410/1451071365179506829/1451155505472077958
不安なところ
その他情報(別で取った議事録等、このプルリクに関連する情報があれば)
ブランチ名について
以下のいずれかをブランチの先頭につけてブランチを命名してください
feature/機能改修bugfix/バグ修正refactor/リファクタリングdeps/依存パッケージなどのアップデートchore/雑用、当てはまるラベルがないときに設定する例:
feature/login-buttonreviewer・mergeについて
for 作業者:
for レビュワー:
※ 作業の担当外の人は勉強になったことなどをコメントできるとBetterです!
レビューのコメントについて
レビュー時はバッヂ(テキストでも可)を付けて、どのレベル感のコメントか明示します。
[must]必ず直すべき[imo]自分の考えは〜[nit]細かい指摘[ask]質問/確認[fyi]ご参考まで例
md

また、基本的に担当者がわからないことを質問することをレビューとします。フロントエンドについては、実際に成果物を触ってチェックするウォークスルーレビューを行います。