-
Notifications
You must be signed in to change notification settings - Fork 3
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
【2人目確認中】[ 投稿リスト ]編集画面では投稿のリンクをクリックしても飛ばないようにしました #2212
base: develop
Are you sure you want to change the base?
Conversation
確認しました |
ありがとうございます!確認しました。編集画面ではリンクが飛ばない方がわたし的には使いやすいのでうれしいです! 私の環境だけかもしれませんが、少しだけ気になる点がありました。 もしかするとわたしの環境の問題かもしれません、もしそうでしたらすみません、、!
実装するかどうかは、石川さんにさらっとご意見いただけたら嬉しいです! -.8083.-.-.-br-.-.-WordPress.mp4 |
@goutetsuguma |
@mtdkei |・w・).oO( 明日でええんやで... ) |
@kurudrive |
リンクされても困るので実装で良いと思いますー。 @mtdkei 「休日だけど対応お願い ( ̄人 ̄)」という特別な指示がなければ休日にメンション飛んできても原則全部無視で勤務時間になってから対応でOKです。 |
@kurudrive |
@goutetsuguma |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#2198
どういう変更をしたか?
投稿リストブロックのリンクをクリック不可にし、カーソルが指にならないように変更しました。
こちらは実際に実装するかどうかも含めて確認してください。
スクリーンショットまたは動画
変更前 Before
2024-09-13.12.30.08.mov
変更後 After
2024-09-13.12.28.39.mov
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
→編集画面でのリンク無効化とスタイル変更に関するDOM操作のためスキップ。
変更内容について何を確認したか、どういう方法で確認をしたかなど
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
実装者と同じ確認をしてください。
こちらは実際に実装するかどうかも含めて確認してください。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。