マウスクリックによるキャレット移動が出来なくなる不具合の解消

[beru]

#547 に対する対策です。

マウス左ボタン押下でキャレットのフォーカスが無い時にも処理を行う事で、マウスによるキャレット移動が出来なくなる不具合を解消します。

既存のコードでマウス左ボタン押下でキャレットのフォーカスが無い時に処理を中断している理由は良く分かっていません。中断しないようにしても特に問題は確認出来ませんでした。何か見過ごしているかもしれませんが…。

[m-tmatma]

再現手順は
#526 (comment)
ですか?

結局再現できたのですか?
こちらでは正確な手順がわからないというのもあり、再現できていません。

再現手順を記載いただけますか?

再現手順を PR にもともとの URL を記載した上で、転記していただけると
確認するのが楽なので助かります。

[m-tmatma]

あと 動画もつけていただけと簡単に確認できるのでうれしいです。

[beru]

再現手順は
#526 (comment)
ですか?

結局再現できたのですか?
こちらでは正確な手順がわからないというのもあり、再現できていません。

再現手順を記載いただけますか?

再現手順を PR にもともとの URL を記載した上で、転記していただけると
確認するのが楽なので助かります。

再現手順は問題を報告してくれた @ds14050 さんと @berryzplus さんが

#526 (comment)
#526 (comment)

に記載してくれました。

自分の方でも再現する事が出来ました。以下のような再現手順です。

• 縦スクロールが有効になるある程度長い文書を用意する
• キャレットの位置が見えない位置にスクロールする
• 他のウィンドウにいったんフォーカスを合わせる
  • 「ファイルを開く」や「検索」や「アウトライン解析」でもOKです。モーダル表示な場合はその後の操作の為にウィンドウを閉じる必要があります。
• 現在のキャレットの位置が見えないスクロール位置はそのままで、マウスの左クリックでキャレット移動や選択を試みる
• 左クリック後に反応が無い。マウスの中ホイールでスクロール操作は可能。キャレットが表示される位置までスクロールすればマウスの左クリック操作が有効になる。

[berryzplus]

LGTMです。

この分岐が入っていた理由として考えられる特殊な設計に心当たりがないわけでもないですが、箇所を洗い出すのは不可能だと思います。常識的にはこういう分岐はしないように思いますので、削ってしまって問題ないと思います。

⇒こういう分岐が必要な特殊な設計がどんなのかというと・・・

• 何かの処理中にユーザ入力をブロックしたいケース
  つまり、マルチスレッド環境を意識したシングルスレッド設計の「時間のかかる処理」です。
  検索とかGrepとかが該当すると思うんですが、分岐が活用されるようなコードは見当たりません。

むしろ逆で、シングルスレッドのイベント処理中に他のイベントを処理させようとするコードがあるくらいで、ユーザ入力を止める、という設計のコードはないように思われます。

じっくり考えても100%結論は出ないので、とりあえず入れてしまう、でよいと思います。
入れた経緯が分からんので削る理由も書きずらいことですし・・・:cry:

[beru]

Merge します。もし問題が見つかったら別の PR で対処しましょう。