Skip to content
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

レコード・カリキュラムの詳細ページで画面端が選択できない問題を解決する #215

Merged
merged 1 commit into from
Jun 28, 2023

Conversation

takumi0616
Copy link
Collaborator

対応Issue

resolve #214

概要

レコードやカリキュラムの詳細ページで画面端が選択できないので、選択できるようにする。

実装内容

原因究明

BackBottonのCSSが原因でdevがページ上のz軸で一番手前にあったので、カーソルが下の要素にアクセスできなかったと思われる

原因解決

BackBottonのボタン部分だけアクセスを許可(pointer-events: auto;)して、他の部分はアクセスできないようにした。(pointer-events: none;)

変更ファイル

  • view/next-project/src/components/common/BackButton/BackButton.module.css

画面スクリーンショット等

テスト項目

  • BackButtonがある全てのページで、BackButtonのボタン部分の中心よりも左側にあるテキストにアクセス(ドラッグ)できる
  • BackButtonがある全てのページで、BackButtonが正常に動く

備考

@vercel
Copy link

vercel bot commented Jun 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nutmeg-seeds ✅ Ready (Inspect) Visit Preview Jun 27, 2023 2:03pm

@YushiroDodo63
Copy link
Collaborator

@takumi0616
↓の3つ、めっちゃよかった!!

  • 実装がスマートでいい!
  • 原因究明から実装→PR作成までが早くてめっちゃいい!
  • スクショもわかりやすくて良い!

それで、↓の3つはGithubだったりPR作る上で気を付けてくれると良いところです!「これがダメだった!」ってことじゃないし別に問題はないけど、プロジェクトのルールだったり知ってた方がいいかな〜ってことを書いただけなんでそこまで気にせず!

  • ブランチの切り方
  • reviewers, assignees に適切な人を入力する
    • 一番下に添付した画像の赤枠の部分の Reviewers には YushiroDodo63 とか imaimai17468 とかを、 Assignees には自分を入力してほしいです!
  • 変更ファイルは書かなくてOK
    • 変更ファイルは、添付画像の緑枠の部分で確認できたりするからPRには書かなくても大丈夫!
スクリーンショット 2023-06-28 11 49 08

Copy link
Collaborator

@YushiroDodo63 YushiroDodo63 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@YushiroDodo63 YushiroDodo63 merged commit dd548ae into develop Jun 28, 2023
@YushiroDodo63 YushiroDodo63 deleted the dev/issue214 branch June 28, 2023 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

レコード・カリキュラムの詳細ページで画面端が選択できない問題を解決する
2 participants