-
Notifications
You must be signed in to change notification settings - Fork 167
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.
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
ツールバー検索ボックスの垂直位置を調整する #771
ツールバー検索ボックスの垂直位置を調整する #771
Conversation
検索ボックスを表示させたときにボックスの下側に大きな隙間ができるので、上下中央になるように位置調整を行います。
レビューぷりーず。 |
欲張りなことを言いますと、「検索ボックスを1ドット下にずらした」とコメントされている「コンボボックスを作る」部分ですね、ここの Y 座標を指定する「rc.top + 1」と高さを指定する「(rc.bottom - rc.top) * 10」を適切な値にするのが根本的な解決方法ではないか、(無駄になる)初期設定をしてからすぐ後で調整するのは今ひとつではないか、という感じを持っています。 sakura/sakura_core/window/CMainToolBar.cpp Line 294 in 4b207b5
|
+1 の高DPI対応はもちろんやってみました。
このコードはドロップダウンリストの高さを指定するものです。 |
だいぶ理解してきました。
CreateWindows の高さ指定が適当な手段ではなく、コンボボックスの垂直位置調整が WM_SETFONT よりあとでなければいけない理由です。
邪魔ではなくともデッドコードです。コメントと +1 を両方削除しておけば無駄に目を引くこともなく、+1 を +2 にしてみようという気を起こさせることもないでしょう。 「検索ボックスを1ドット下にずらした」という調整が「コンボボックスの垂直位置を調整する」によって発展的に置き換えられたと考えるなら消すのが当然と思います。 |
ツールバーボタン矩形に対して1px下にずらして作成する記述になっていた。コンボボックスの実際の高さはフォントを指定したあとに決まるので、位置調整は作ったあとに行うようにして作成時の位置調整は削っておく。
コメントと+1の件、ようやくコミット積めました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
追加コミットありがとうございます。DPI 対応と同時にフォントサイズの変更にも対応できているのがいいですね。
レビューありがとうございます。 |
…earchbox ツールバー検索ボックスの垂直位置を調整する
検索ボックスを表示させたときにボックスの下側に大きな隙間ができるので、上下中央になるように位置調整を行います。
キャプチャは200%表示です。
表示倍率が高いほど下端の余白が大きくなります。
(4Kディスプレイで最大表示(=350%)すると大変なことに・・・。)
周囲のボタンと上端が揃うように位置調整していますが、
下端が揃うほうがいいという人がいるかも知れません。
細かい調整の話はおいておいて、まずは位置調整の機構を入れたいです。