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

ツールバーの検索ボックスにフォーカスを当てていないにも関わらず文字列が選択状態として表示される現象が起きないように対策 #1724

Merged
merged 1 commit into from
Sep 8, 2021
Merged

ツールバーの検索ボックスにフォーカスを当てていないにも関わらず文字列が選択状態として表示される現象が起きないように対策 #1724

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
13 changes: 8 additions & 5 deletions sakura_core/window/CMainToolBar.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -293,6 +293,9 @@ void CMainToolBar::CreateToolBar( void )
//位置とサイズを取得する
rc.right = rc.left = rc.top = rc.bottom = 0;
Toolbar_GetItemRect( m_hwndToolBar, count-1, &rc );
// Social Distance
rc.left += cxBorder;
rc.right -= cxBorder;
Copy link
Contributor

Choose a reason for hiding this comment

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

Social Distance

時事用語ですね。beforeコロナ時代にあった「不適切な密着状態」を戒めるコトバですな。
ツールバーのボタン同士に人間社会が形成されているとは考えづらいので誤用と考えられます。
まぁ、まともな理解力を持った人にはニュアンス伝わると思うのでこのままでよいと思いますが。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ここは意図的にこの言葉を使いました。記念として…。


//コンボボックスを作る
m_hwndSearchBox = CreateWindow( WC_COMBOBOX, L"Combo",
Expand Down Expand Up @@ -334,15 +337,15 @@ void CMainToolBar::CreateToolBar( void )

CDialog::SetComboBoxDeleter(m_hwndSearchBox, &m_cRecentSearch);

// コンボボックスの位置と幅を調整する
// コンボボックスの垂直位置を調整する
CMyRect rcCombo;
::GetWindowRect( m_hwndSearchBox, &rcCombo );
::SetWindowPos( m_hwndSearchBox, NULL,
rc.left + cxBorder,
rc.left, //作ったときと同じ値を指定
rc.top + (rc.bottom - rc.top - rcCombo.Height()) / 2,
rcCombo.Width() - cxBorder * 2,
rcCombo.Height(),
SWP_NOZORDER | SWP_NOACTIVATE | SWP_NOOWNERZORDER | SWP_NOSENDCHANGING );
0, //rcCombo.Width()のまま変えない
0, //rcCombo.Height()のまま変えない
SWP_NOSIZE | SWP_NOZORDER | SWP_NOACTIVATE | SWP_NOOWNERZORDER | SWP_NOSENDCHANGING );
Copy link
Contributor

Choose a reason for hiding this comment

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

SWP_NOSIZE |

ここが変更のキモでしょうか。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

そうですね、何故かウィンドウ作成後にサイズを変えるとテキスト選択状態の見た目になってしまうという。。
他にも色々と条件がありそうですが、まぁあんまり考えたくないです。。

}
break;

Expand Down