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

モバイルでの投稿モーダルで、投稿しようと思って閉じてしまいやすい #14051

Open
1 task
fruitriin opened this issue Jun 20, 2024 · 15 comments
Labels
⚠️bug? This might be a bug packages/frontend Client side specific issue/PR

Comments

@fruitriin
Copy link
Contributor

💡 Summary

投稿モーダルの投稿ボタンを押そうとして意図せずモーダルが閉じてしまうことがある

🥰 Expected Behavior

投稿したいときにしくじらずに投稿できる

🤬 Actual Behavior

投稿ボタンを押そうとした時に、モーダルの外をクリックしたときのモーダルを閉じる操作に判定が吸われることがある
これはモバイル端末のようなタッチデバイスで顕著である
IMG_2199

📝 Steps to Reproduce

No response

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@fruitriin fruitriin added the ⚠️bug? This might be a bug label Jun 20, 2024
@fruitriin
Copy link
Contributor Author

fruitriin commented Jun 20, 2024

解決案1. モバイル端末の場合投稿モーダルの余白をなくす
Pros. Twitterのように投稿という一つのモードに突入する
Cons. レイアウトが変わる
解決案2. 投稿モーダルの場合、明示的に閉じるボタンを押すかESCしたとき以外、クリックでモーダルの外をクリックしても閉じないようにする
Pros. 今までのレイアウトを維持
Cons. 挙動が変わる
解決案3. 投稿ボタンの裏に、当たり判定が消えないようにより大きな透明なボックスを配置する
Pros. 現行を踏襲しつついいとこどり
Cons. ハックっぽい

@tai-cha
Copy link
Member

tai-cha commented Jun 20, 2024

逆にモーダル外タップでキャンセルする派なのでその仕様変更すると慣れるまでは×ボタン押さないといけないので大変かもしれない(×ボタンを押せばいいと言えばそうだけれど)

@anatawa12
Copy link
Member

私はハックっぽいけど捜査感の大きな差が出なさそうな3に賛成

@syuilo
Copy link
Member

syuilo commented Jun 21, 2024

Xはどうなってるのかしら

@KisaragiEffective
Copy link
Sponsor Member

Xはどうなってるのかしら

image

全画面表示になって誤タップできない

@syuilo
Copy link
Member

syuilo commented Jun 21, 2024

その方が良いかも

@anatawa12
Copy link
Member

anatawa12 commented Jun 21, 2024

Xとかの一番上のキャンセルを押さないといけないのは、最近の縦長のスマホでは上の方に指伸ばさないと閉じれないので結構体験は悪いので下の方で閉じれるのは維持してほしい気持ちがある

image

image

@Sayamame-beans
Copy link
Member

Android環境においても、端末下部の戻るボタンを押すと投稿フォームが閉じるのではなくMisskey PWAごと閉じて/戻ってしまうので、全画面表示はあまり体験が良くないかもです。
(off-topic: この"戻るボタンでPWAごと閉じてしまう"挙動、前にissueにしたつもりでしたがしてなさそうなので後で書きます)

@syuilo
Copy link
Member

syuilo commented Jun 21, 2024

戻るボタン押すとフォームが閉じるようにすれば良さそう

@anatawa12
Copy link
Member

ios、戻るボタンがないので困る気がします

@syuilo
Copy link
Member

syuilo commented Jun 21, 2024

無い分には困らなくないかしら

@fruitriin
Copy link
Contributor Author

戻るボタンはデバイスの戻るボタンの話で、デバイスの戻るボタンを押すと前のページではなくPWAが閉じてしまう挙動のことだとおもうので、iOSこの戻るボタンがないので特に困らないとおもいます

@anatawa12
Copy link
Member

anatawa12 commented Jun 21, 2024

遠い問題がまだ困ると思う(戻るボタンがドロは下にあるのでそれで解決できるけど)

@Sayamame-beans Sayamame-beans added the packages/frontend Client side specific issue/PR label Jun 21, 2024
@KisaragiEffective
Copy link
Sponsor Member

折衷案: 上、左、右のマージンは消すが下は現状維持

@fruitriin
Copy link
Contributor Author

下は空いてるのいいよね

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug packages/frontend Client side specific issue/PR
Projects
None yet
Development

No branches or pull requests

6 participants