-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
モバイルでの投稿モーダルで、投稿しようと思って閉じてしまいやすい #14051
Comments
解決案1. モバイル端末の場合投稿モーダルの余白をなくす |
逆にモーダル外タップでキャンセルする派なのでその仕様変更すると慣れるまでは×ボタン押さないといけないので大変かもしれない(×ボタンを押せばいいと言えばそうだけれど) |
私はハックっぽいけど捜査感の大きな差が出なさそうな3に賛成 |
Xはどうなってるのかしら |
その方が良いかも |
Android環境においても、端末下部の戻るボタンを押すと投稿フォームが閉じるのではなくMisskey PWAごと閉じて/戻ってしまうので、全画面表示はあまり体験が良くないかもです。 |
戻るボタン押すとフォームが閉じるようにすれば良さそう |
ios、戻るボタンがないので困る気がします |
無い分には困らなくないかしら |
戻るボタンはデバイスの戻るボタンの話で、デバイスの戻るボタンを押すと前のページではなくPWAが閉じてしまう挙動のことだとおもうので、iOSこの戻るボタンがないので特に困らないとおもいます |
遠い問題がまだ困ると思う(戻るボタンがドロは下にあるのでそれで解決できるけど) |
折衷案: 上、左、右のマージンは消すが下は現状維持 |
下は空いてるのいいよね |
💡 Summary
投稿モーダルの投稿ボタンを押そうとして意図せずモーダルが閉じてしまうことがある
🥰 Expected Behavior
投稿したいときにしくじらずに投稿できる
🤬 Actual Behavior
投稿ボタンを押そうとした時に、モーダルの外をクリックしたときのモーダルを閉じる操作に判定が吸われることがある
![IMG_2199](https://private-user-images.githubusercontent.com/18308639/341437643-4d936c85-267a-4714-ac4b-7f2732d857d5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA3NjA4MDAsIm5iZiI6MTcyMDc2MDUwMCwicGF0aCI6Ii8xODMwODYzOS8zNDE0Mzc2NDMtNGQ5MzZjODUtMjY3YS00NzE0LWFjNGItN2YyNzMyZDg1N2Q1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzEyVDA1MDE0MFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTc4MzRjYzE1Y2RmZDI4MGM0Mjk4NWYzNDM0ZDVjMmQ0YWJkZWUxYWRiMzlmODgzYmZlMWI1YmY0MmYxNTU1ZjAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.FYgPpNZ5wSO_I3ydDM0rL5LIzCy6EQZfhZiEyP2GjnY)
これはモバイル端末のようなタッチデバイスで顕著である
📝 Steps to Reproduce
No response
💻 Frontend Environment
🛰 Backend Environment (for server admin)
Do you want to address this bug yourself?
The text was updated successfully, but these errors were encountered: