utubo / firefox-simple_gesture

🦊An add-on that adds simple touch gestures for Firfox for Android. Sorry, I only check issues sometimes.
https://addons.mozilla.org/ja/firefox/addon/simple-gesture/
Other
29 stars 1 forks source link

Custom gesturesを保存したときの画面遷移がおかしい #37

Closed h-youhei closed 6 years ago

h-youhei commented 6 years ago

simple_gesture 2.4 android 4.4.2 firefox 58.0.1

Scriptを編集して保存すると、New tab のところに飛びます。 Custom gestures のところに飛んでほしいです。

Scriptを編集せずに保存するとスクロールは起こりません。

h-youhei commented 6 years ago

画面遷移がおかしいのはScriptにエラーがあるときだけでした。

utubo commented 6 years ago

報告ありがとうございます現象確認しました。 近日中にちょっと強引に対処してみます。 ダイアログを表示したときのスクロール位置を覚えておいて、クローズしたら戻すという対応になりそうです。 (ダイアログを閉じる等のスクロール位置はFirefox任せにしていた)

原因わかったっぽいのでまとめ

保存ボタン等を押す ↓ テキストエリアからフォーカスが外れキーボードが閉じ始める(アニメーションあり) ↓ history.back()でスクロール位置が戻る ↓ キーボードのクローズが完了する ↓ キーボードの高さ分だけ上へスクロールする

メモ

utubo commented 6 years ago

修正してみました。(ver. 2.4.1) 各ページの高さの指定にmin-height: 100vhを指定していたのが問題だったみたいなので、固定pxを設定するようにしました。 画面を回転させたときにページの高さが更新されなくなるけれど、そこは断念。

h-youhei commented 6 years ago

修正確認しました。ありがとうございます、

utubo commented 6 years ago

よかった ありがとうございました。助かりました。

utubo commented 6 years ago

原因まとめ https://twitter.com/utbutb/statuses/959043720367624192