fix(frontend): ユーザー・ノート検索の確定挙動をformタグ経由で行うように

misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀

https://misskey-hub.net/

GNU Affero General Public License v3.0

9.95k stars 1.35k forks source link

Open kakkokari-gtyih opened 3 weeks ago

kakkokari-gtyih commented 3 weeks ago

type=buttonを指定しないとsubmitのボタンとして扱われるのでそのへんの改変も含んでいる

Fix #14570

syuilo commented 3 weeks ago

うーーーーむ

syuilo commented 3 weeks ago

formに頼るのは無理が生じてきそう(formは古典的な文書寄りのHTML向けの設計だけど、Misskey WebはWebアプリ寄りのため)

syuilo commented 3 weeks ago

実際このPRでもボタンとは言い難いものをbuttonにしなければいけなくなっているし

syuilo commented 3 weeks ago

あと使えるタグに制限が生じそうな気もする

syuilo commented 3 weeks ago

そこら辺はformタグの仕様を調べたりする必要があるけど、めんどくさいから大人しくdiv使っといてJSで制御した方が良さそう

kakkokari-gtyih commented 3 weeks ago

実際このPRでもボタンとは言い難いものをbuttonにしなければいけなくなっているし

MkFoldableSectionのボタンの場所を変えたのはどっちかというとフォーカスの当たり方の調整（単にボタンの誤作動を防ぐだけでいいのならもともとbuttonだったところにtype=buttonを入れるだけで良い）

kakkokari-gtyih commented 3 weeks ago

大人しくdiv使っといてJSで制御した方が良さそう

スマホのキーボードの確定ボタンに対応するイベントあるのかしら

syuilo commented 3 weeks ago

enterイベントだわね

kakkokari-gtyih commented 3 weeks ago

enterイベントだわね

元issueによるとそれが動作しないとのことなのでこういうアプローチにしたんだけど…

syuilo commented 3 weeks ago

謎

syuilo commented 3 days ago

form内で使ってるコンポーネントが内部的にbutton使ってるとそれ押すとsubmtされちゃう(そして親が子コンポーネントの内部に干渉してtype=buttonを付けたりはできないから回避不能)からやっぱり現実的ではないかも