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.

Sign up for GitHub

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

プロパティシートのコントロール表示のちらつき防止 #1424

Merged
merged 2 commits into from
Oct 11, 2020
Merged

プロパティシートのコントロール表示のちらつき防止 #1424

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1a1408b
プロパティシートのコントロール表示のちらつき防止
beru Oct 4, 2020
079929f
SendMessage anywayz
beru Oct 5, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions sakura_core/prop/CPropComCustmenu.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -198,13 +198,13 @@ INT_PTR CPropCustmenu::DispatchEvent(
nIdx3 = Combo_GetCurSel( hwndCOMBO_FUNCKIND );
nIdx4 = List_GetCurSel( hwndLIST_FUNC );
if( nIdx1 != CB_ERR ){
::SendMessageAny( hwndDlg, WM_COMMAND, MAKEWPARAM( IDC_COMBO_MENU, CBN_SELCHANGE ), (LPARAM)hwndCOMBO_MENU );
::SendMessage( hwndDlg, WM_COMMAND, MAKEWPARAM( IDC_COMBO_MENU, CBN_SELCHANGE ), (LPARAM)hwndCOMBO_MENU );
if( nIdx2 != LB_ERR ){
List_SetCurSel( hwndLIST_RES, nIdx2 );
}
}
if( nIdx3 != CB_ERR ){
::SendMessageAny( hwndDlg, WM_COMMAND, MAKEWPARAM( IDC_COMBO_FUNCKIND, CBN_SELCHANGE ), (LPARAM)hwndCOMBO_FUNCKIND );
::SendMessage( hwndDlg, WM_COMMAND, MAKEWPARAM( IDC_COMBO_FUNCKIND, CBN_SELCHANGE ), (LPARAM)hwndCOMBO_FUNCKIND );
if( nIdx4 != LB_ERR ){
List_SetCurSel( hwndLIST_FUNC, nIdx4 );
}
Expand Down Expand Up @@ -665,6 +665,7 @@ void CPropCustmenu::SetDataMenuList(HWND hwndDlg, int nIdx)
WCHAR szLabel[300];
WCHAR szLabel2[300+4];

::SendMessage( hwndDlg, WM_SETREDRAW, FALSE, 0 );
/* メニュー項目一覧に文字列をセット（リストボックス）*/
HWND hwndLIST_RES = ::GetDlgItem( hwndDlg, IDC_LIST_RES );
// hwndEDIT_KEY = ::GetDlgItem( hwndDlg, IDC_EDIT_KEY );
Expand Down Expand Up @@ -694,6 +695,8 @@ void CPropCustmenu::SetDataMenuList(HWND hwndDlg, int nIdx)
::DlgItem_SetText( hwndDlg, IDC_EDIT_MENUNAME, m_Common.m_sCustomMenu.m_szCustMenuNameArr[nIdx] );

CheckDlgButtonBool( hwndDlg, IDC_CHECK_SUBMENU, m_Common.m_sCustomMenu.m_bCustMenuPopupArr[nIdx] );
::SendMessage( hwndDlg, WM_SETREDRAW, TRUE, 0 );
::InvalidateRect( hwndDlg, NULL, FALSE );
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ん？これは必要ですか・・・？
（たぶん、必要だから入れてると思いますが念のための確認です。）

window描画の仕組み

  1. 外部から WM_UPDATE を送る。
  2. WM_UPDATE を受け取ったウインドウは、クライアント領域に更新領域があれば自身に WM_PAINT を送る。
  3. WM_PAINT を受け取ったウインドウは、クライアント領域を描画する。

InvalidateRect(hwnd, NULL, FALSE) は、クライアント領域の全域更新を指示する命令です。WM_SETREDRAW で再描画フラグを戻しただけだけだと更新されない仕様だと思うんですが、ここに要るなら他にもいるような気がして、何故ここだけ？と思いました。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コントロールのウィンドウの変更に対して WM_SETREDRAW する場合は後で InvalidateRect を呼ばないでも表示が更新されてたんですが、親のダイアログの場合は呼ばないと表示が更新されませんでした。理屈は分かっていません。

return;
}

Expand Down
2 changes: 2 additions & 0 deletions sakura_core/prop/CPropComKeybind.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -460,6 +460,7 @@ void CPropKeybind::ChangeKeyList( HWND hwndDlg){
i |= _ALT;
wcscat( szKeyState, L"Alt+" );
}
::SendMessage( hwndKeyList, WM_SETREDRAW, FALSE, 0 );
/* キー一覧に文字列をセット（リストボックス）*/
List_ResetContent( hwndKeyList );
for( i = 0; i < m_Common.m_sKeyBind.m_nKeyNameArrNum; ++i ){
Expand All @@ -470,6 +471,7 @@ void CPropKeybind::ChangeKeyList( HWND hwndDlg){
List_SetCurSel( hwndKeyList, nIndex );
List_SetTopIndex( hwndKeyList, nIndexTop );
::SendMessageCmd( hwndDlg, WM_COMMAND, MAKELONG( IDC_LIST_KEY, LBN_SELCHANGE ), (LPARAM)hwndKeyList );
::SendMessage( hwndKeyList, WM_SETREDRAW, TRUE, 0 );
}

/* Keybind:キー割り当て設定をインポートする */
Expand Down
11 changes: 5 additions & 6 deletions sakura_core/prop/CPropComKeyword.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -616,6 +616,8 @@ void CPropKeyword::SetKeyWordSet( HWND hwndDlg, int nIdx )
HWND hwndList;
LV_ITEM lvi;

::SendMessage( hwndDlg, WM_SETREDRAW, FALSE, 0 );

ListView_DeleteAllItems( ::GetDlgItem( hwndDlg, IDC_LIST_KEYWORD ) );
if( 0 <= nIdx ){
::EnableWindow( ::GetDlgItem( hwndDlg, IDC_BUTTON_DELSET ), TRUE );
Expand Down Expand Up @@ -654,9 +656,6 @@ void CPropKeyword::SetKeyWordSet( HWND hwndDlg, int nIdx )
nNum = m_Common.m_sSpecialKeyword.m_CKeyWordSetMgr.GetKeyWordNum( nIdx );
hwndList = ::GetDlgItem( hwndDlg, IDC_LIST_KEYWORD );

// 2005.01.25 Moca/genta リスト追加中は再描画を抑制してすばやく表示
::SendMessageAny( hwndList, WM_SETREDRAW, FALSE, 0 );

for( i = 0; i < nNum; ++i ){
/* ｎ番目のセットのｍ番目のキーワードを返す */
const WCHAR* pszKeyWord = m_Common.m_sSpecialKeyword.m_CKeyWordSetMgr.GetKeyWord( nIdx, i );
Expand All @@ -670,12 +669,12 @@ void CPropKeyword::SetKeyWordSet( HWND hwndDlg, int nIdx )
}
m_Common.m_sSpecialKeyword.m_CKeyWordSetMgr.m_nCurrentKeyWordSetIdx = nIdx;

// 2005.01.25 Moca/genta リスト追加完了のため再描画許可
::SendMessageAny( hwndList, WM_SETREDRAW, TRUE, 0 );

//キーワード数を表示する。
DispKeywordCount( hwndDlg );

::SendMessage( hwndDlg, WM_SETREDRAW, TRUE, 0 );
InvalidateRect(hwndDlg, NULL, FALSE);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここにはありますね・・・

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここも親ダイアログに対しての場合ですね。


return;
}

Expand Down
2 changes: 2 additions & 0 deletions sakura_core/prop/CPropComMainMenu.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -417,6 +417,7 @@ INT_PTR CPropMainMenu::DispatchEvent(
case CBN_SELCHANGE:
nIdxFIdx = Combo_GetCurSel( hwndComboFunkKind );

::SendMessage( hwndListFunk, WM_SETREDRAW, FALSE, 0 );
if (nIdxFIdx == nSpecialFuncsNum) {
// 機能一覧に特殊機能をセット
List_ResetContent( hwndListFunk );
Expand All @@ -428,6 +429,7 @@ INT_PTR CPropMainMenu::DispatchEvent(
/* 機能一覧に文字列をセット（リストボックス）*/
m_cLookup.SetListItem( hwndListFunk, nIdxFIdx );
}
::SendMessage( hwndListFunk, WM_SETREDRAW, TRUE, 0 );

return TRUE;
}
Expand Down
4 changes: 4 additions & 0 deletions sakura_core/prop/CPropComTab.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,6 +191,8 @@ INT_PTR CPropTab::DispatchEvent( HWND hwndDlg, UINT uMsg, WPARAM wParam, LPARAM
/* ダイアログデータの設定 */
void CPropTab::SetData( HWND hwndDlg )
{
::SendMessage(hwndDlg, WM_SETREDRAW, (WPARAM)FALSE, 0);

// Feb. 11, 2007 genta「ウィンドウ」シートより移動
::CheckDlgButton( hwndDlg, IDC_CHECK_DispTabWnd, m_Common.m_sTabBar.m_bDispTabWnd ); //@@@ 2003.05.31 MIK
::CheckDlgButton( hwndDlg, IDC_CHECK_SameTabWidth, m_Common.m_sTabBar.m_bSameTabWidth ); //@@@ 2006.01.28 ryoji
Expand Down Expand Up @@ -233,6 +235,8 @@ void CPropTab::SetData( HWND hwndDlg )
m_hTabFont = SetFontLabel( hwndDlg, IDC_STATIC_TABFONT, m_Common.m_sTabBar.m_lf, m_Common.m_sTabBar.m_nPointSize);

EnableTabPropInput(hwndDlg);

::SendMessage(hwndDlg, WM_SETREDRAW, (WPARAM)TRUE, 0);
}

/* ダイアログデータの取得 */
Expand Down
2 changes: 2 additions & 0 deletions sakura_core/prop/CPropComToolbar.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -268,6 +268,7 @@ INT_PTR CPropToolbar::DispatchEvent(
case CBN_SELCHANGE:
nIndex2 = Combo_GetCurSel( hwndCombo );

::SendMessage( hwndFuncList, WM_SETREDRAW, FALSE, 0 );
List_ResetContent( hwndFuncList );

/* 機能一覧に文字列をセット (リストボックス) */
Expand All @@ -286,6 +287,7 @@ INT_PTR CPropToolbar::DispatchEvent(
lResult = List_SetItemHeight( hwndFuncList, lResult, nListItemHeight );
}
}
::SendMessage( hwndFuncList, WM_SETREDRAW, TRUE, 0 );
return TRUE;
}
}else{
Expand Down
8 changes: 4 additions & 4 deletions sakura_core/sakura_rc.rc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1173,7 +1173,7 @@ END

IDD_PROP_TAB DIALOGEX 0, 0, 293, 240
STYLE DS_MODALFRAME | DS_3DLOOK | DS_CENTER | DS_CONTEXTHELP | WS_POPUP | WS_CAPTION | WS_SYSMENU
EXSTYLE WS_EX_CONTEXTHELP
EXSTYLE WS_EX_CONTEXTHELP | WS_EX_COMPOSITED
FONT 9, "ＭＳ Ｐゴシック"
BEGIN
CONTROL "タブバーを表示する(&D)", IDC_CHECK_DispTabWnd, "Button", BS_AUTOCHECKBOX | WS_TABSTOP, 7, 11, 111, 10
Expand All @@ -1187,7 +1187,7 @@ BEGIN
LTEXT "閉じるボタン(&X)", IDC_TextTabClose, 124, 116, 48, 8
COMBOBOX IDC_CHECK_DispTabClose, 173, 113, 46, 80, CBS_DROPDOWNLIST | WS_VSCROLL | WS_TABSTOP
PUSHBUTTON "フォント(&F)...", IDC_BUTTON_TABFONT, 232, 113, 51, 14
RTEXT "Font", IDC_STATIC_TABFONT, 102, 127, 180, 17, SS_RIGHT
RTEXT "Font", IDC_STATIC_TABFONT, 102, 127, 180, 12, SS_RIGHT
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これは必要です？ or NOT？

必要なら入れたらよいと思います。
右寄せラベルの高さが大きくなってるのを文字に合わせる変更に見えていて、ちらつき防止と関係あるようには思えないのですが・・・。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WS_EX_COMPOSITED を付けるとこの変更入れないと表示に問題がありました。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

高さを調整しないとこのように表示されてしまいます。

Copy link
Contributor Author

@beru beru Oct 5, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://docs.microsoft.com/en-us/windows/win32/winmsg/extended-window-styles

の WS_EX_COMPOSITED の解説ですが

Paints all descendants of a window in bottom-to-top painting order using double-buffering. Bottom-to-top painting order allows a descendent window to have translucency (alpha) and transparency (color-key) effects, but only if the descendent window also has the WS_EX_TRANSPARENT bit set. Double-buffering allows the window and its descendents to be painted without flicker. This cannot be used if the window has a class style of either CS_OWNDC or CS_CLASSDC.
Windows 2000: This style is not supported.

と書かれています。bottom-to-top painting order というのが画面のXYのYの上下の事なのかZ-orderの上下の事なのかわかっていませんが（多分後者）この事が関わっていそうです。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bottom-to-top painting order というのが画面のXYのYの上下の事なのかZ-orderの上下の事なのかわかっていませんが（多分後者）この事が関わっていそうです。

画面の下から上の順番でコントロールを描画していくってことでしょうね。
通常は上から下の順番で描画されていると考えると、下方向にはみ出したコントロールがこれまで問題にならず、拡張スタイルの追加でおかしくなった理由を説明できると思います。従来ははみ出した部分を下側にあるコントロールで上書きしていたから問題にならず、変更により描画済みのコントロールの上にはみ出して描画するようになったっちゅうことです。

CONTROL "タブ一覧をソートする(&S)", IDC_CHECK_SortTabList, "Button", BS_AUTOCHECKBOX | WS_TABSTOP, 15, 129, 85, 10
CONTROL "多段(&M)", IDC_CHECK_TAB_MULTILINE, "Button", BS_AUTOCHECKBOX | WS_TABSTOP, 15, 141, 60, 10
RTEXT "表示位置(&P)", IDC_TAB_POSITION, 80, 143, 40, 8
Expand Down Expand Up @@ -1404,7 +1404,7 @@ END

IDD_PROP_KEYBIND DIALOGEX 0, 0, 293, 240
STYLE DS_MODALFRAME | DS_CONTEXTHELP | WS_POPUP | WS_CAPTION | WS_SYSMENU
EXSTYLE WS_EX_CONTEXTHELP
EXSTYLE WS_EX_CONTEXTHELP | WS_EX_COMPOSITED
FONT 9, "ＭＳ Ｐゴシック"
BEGIN
RTEXT "種別(&K)", IDC_LABEL_MENUFUNCKIND, 4, 4, 51, 8
Expand All @@ -1428,7 +1428,7 @@ END

IDD_PROP_CUSTMENU DIALOGEX 0, 0, 293, 240
STYLE DS_MODALFRAME | WS_POPUP | WS_CAPTION | WS_SYSMENU
EXSTYLE WS_EX_CONTEXTHELP
EXSTYLE WS_EX_CONTEXTHELP | WS_EX_COMPOSITED
FONT 9, "ＭＳ Ｐゴシック"
BEGIN
RTEXT "種別(&K)", IDC_LABEL_MENUFUNCKIND, 4, 4, 52, 8
Expand Down