ツールバーの表示ON/OFFを行うとちらつく現象への対策

[beru]

変更前	変更後

[beru]

なぜこの変更を入れるとちらつく現象が改善するのかメカニズムは分かっていません…。

[beru]

この PR 用にスクリーンキャプチャした動画のコマを確認している際に気付いた点ですが、複数のサクラエディタのウィンドウを開いている状態で Ctrl + 1 押しでツールバーを表示から非表示に切り替えるとフォーカスが合っていないウィンドウのツールバーも非表示になりますが、その際にアイコンが消えてからツールバーが消えています。本来は同時に消えるべきだと思います。フォーカスが合っているウィンドウは問題有りません。一瞬の出来事なので気付きにくいですが直した方が表示切替がすっきり動くと思います。

[m-tmatma]

問題が発生するときの m_nWinSizeType の値は何になっていますか?
(https://docs.microsoft.com/en-us/windows/desktop/winmsg/wm-size の wParam)

[beru]

m_nWinSizeType の値は 0 になっている事を確認しました。

[berryzplus]

これたぶん、前回の WM_SIZE で送られてきた wParam のキャッシュだと思います。

最近のwindowsのサイズ変更シーケンスは複雑なので、
ここのキャッシュに意味があるかどうか結構微妙だと思っとります。
SIZE_RESTORED(=0) の決め打ちで指定してやってもいいような気がします。

[berryzplus]

なぜこの変更を入れるとちらつく現象が改善するのかメカニズムは分かっていません…。

分かってないと言っちゃうわけですねw

本質的には CEditWnd::OnSize2 の問題だと思っとります。
どっかで CEditWnd::OnSize2 自体に手を入れたいなと考えてます。
前にどっかで書いたけど、これめちゃめちゃ効率悪い処理なので。

WM_SIZE は 実際にはウインドウサイズを変えないので、
これを挟むことで改善する事象があるなら入れていいと思います。

SendMessage(hWnd, WM_SIZE, 0, 0)で改善するってことは、
ウインドウサイズが変わらない場合の処理に問題がありそうな気配です。
いま詳細を追求する意味をあまり感じないので、後日でいいと思ってます。

[berryzplus]

LGTMです。

暫定対処の効果を確認できたので入れてしまってよいと考えます。

[beru]

この対策ですがタブバーを表示していない時でも（ツールバーの表示ON/OFF時に）確認出来る横スクロールバーのちらつき（というか残像というか…）にも効果がありますね。

変更前	変更後

変更後 はタブバーを表示していない時でも同様の処理をするように、

if( m_pShareData->m_Common.m_sTabBar.m_bDispTabWnd ){

の判定を外したものです。

[beru]

分かってないと言っちゃうわけですねw

本質的には CEditWnd::OnSize2 の問題だと思っとります。
どっかで CEditWnd::OnSize2 自体に手を入れたいなと考えてます。
前にどっかで書いたけど、これめちゃめちゃ効率悪い処理なので。

WM_SIZE は 実際にはウインドウサイズを変えないので、
これを挟むことで改善する事象があるなら入れていいと思います。

SendMessage(hWnd, WM_SIZE, 0, 0)で改善するってことは、
ウインドウサイズが変わらない場合の処理に問題がありそうな気配です。
いま詳細を追求する意味をあまり感じないので、後日でいいと思ってます。

CEditWnd::OnSize2 を見てみましたが何だか色々な事をやっていますね。変更すると色々な動作環境で思わぬ不具合が出たりしないか心配になっちゃいます…。MoveWindow を複数回呼んでいますが、BeginDeferWindowPos, DeferWindowPos, EndDeferWindowPos で一括処理した方が良いんでしょうね。

[berryzplus]

CEditWnd::OnSize2 を見てみましたが何だか色々な事をやっていますね。変更すると色々な動作環境で思わぬ不具合が出たりしないか心配になっちゃいます…。

色んなことやり過ぎ＋キャッシュ変数大杉なので、
不具合を起こしやすいコードだと思います。
本体を触らずに軽減できる方法があるのだから、とりあえずそれで対応でいいような気がしています。

MoveWindow を複数回呼んでいますが、BeginDeferWindowPos, DeferWindowPos, EndDeferWindowPos で一括処理した方が良いんでしょうね。

おそらく歴史的にちょっとずつ増えていったものなので、
一括処理になってないのは仕方ないのかなとか思っています。
位置決めロジックと位置変更ロジックは分離ができるはずなので、
うまいことどうにかならんかなぁ・・・と思っています。

[beru]

この暫定対処ですが、タブバーを表示していない時でもツールバーのON/OFF時に発生する横スクロールバーのちらつきにも効果が有るので、タブバーを表示していない時でも実施するように変更しました。

再度レビューお願いします。

[berryzplus]

LGTMです。

SendMessage( GetHwnd(), WM_SIZE, 0, 0) が何らかのステータスをリセットしている感じなんですかね。裏が取れてるならリネームお願いするんですけど、今回は空気でやってるのでこのままいっちゃいましょう。

[beru]

それでは Merge します。もし問題が見つかったら別の PR で対処する事にしましょう。