Skip to content

Conversation

@TkymHrt
Copy link
Member

@TkymHrt TkymHrt commented Apr 15, 2024

対応Issue

resolve #730

概要

モーダルを開いた際に、閉じるボタンが押せなくなる問題を修正
モーダルの縦幅を90vhに制限し、以降は内部でスクロールする仕組みに変更しました。
OpenModalButtonにCSSを当てず、AddButtonに直接記述する方式に切り替えました。

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

FireShot Capture 001 - 協賛活動一覧 - localhost FireShot Capture 002 - 協賛活動一覧 - localhost
FireShot Capture 003 - 協賛活動一覧 - localhost FireShot Capture 004 - 協賛活動一覧 - localhost

テスト項目

  • localhost:3000/sponsoractivitiesにアクセスできる。
  • 編集モーダル・修正モーダル・新規登録のモーダルが画面幅を超過すると、モーダル内部でスクロールできる。
  • Z-Indexが正常に働いている。→ ヘッダーと重ならないかどうか。

備考

@TkymHrt TkymHrt self-assigned this Apr 15, 2024
@github-actions github-actions bot added the bug Something isn't working label Apr 15, 2024
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

少しコメントしましたー

…lButton.tsx

Co-authored-by: Kubosaka <115447919+Kubosaka@users.noreply.github.com>
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

LGTM

@Kubosaka Kubosaka merged commit e4abfbd into develop May 14, 2024
@Kubosaka Kubosaka deleted the fix/yama/730-make-modal-responsive branch May 14, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

協賛活動の登録・編集モーダルのレスポンシブ対応

3 participants