ツールバー検索ボックスの垂直位置を調整する

[berryzplus]

検索ボックスを表示させたときにボックスの下側に大きな隙間ができるので、上下中央になるように位置調整を行います。

...	画像
変更前
変更後

キャプチャは200%表示です。
表示倍率が高いほど下端の余白が大きくなります。
(4Kディスプレイで最大表示(=350%)すると大変なことに・・・。)

周囲のボタンと上端が揃うように位置調整していますが、
下端が揃うほうがいいという人がいるかも知れません。
細かい調整の話はおいておいて、まずは位置調整の機構を入れたいです。

[berryzplus]

レビューぷりーず。

[ds14050]

欲張りなことを言いますと、「検索ボックスを1ドット下にずらした」とコメントされている「コンボボックスを作る」部分ですね、ここの Y 座標を指定する「rc.top + 1」と高さを指定する「(rc.bottom - rc.top) * 10」を適切な値にするのが根本的な解決方法ではないか、(無駄になる)初期設定をしてからすぐ後で調整するのは今ひとつではないか、という感じを持っています。

sakura/sakura_core/window/CMainToolBar.cpp

Line 294 in 4b207b5

// Mar. 8, 2003 genta 検索ボックスを1ドット下にずらした

[berryzplus]

欲張りなことを言いますと、「検索ボックスを1ドット下にずらした」とコメントされている「コンボボックスを作る」部分ですね、ここの Y 座標を指定する「rc.top + 1」と高さを指定する「(rc.bottom - rc.top) * 10」を適切な値にするのが根本的な解決方法ではないか、(無駄になる)初期設定をしてからすぐ後で調整するのは今ひとつではないか、という感じを持っています。

+1 の高DPI対応はもちろんやってみました。
しかし、期待した効果は得られませんでした。
あっても邪魔はしないので残していますが、削ったほうがよければ削ります。

「(rc.bottom - rc.top) * 10」

このコードはドロップダウンリストの高さを指定するものです。
ツールバーボタンの高さを基準にした相対指定なので変えなくていいと思っています。
「履歴をもっとたくさん表示してほしい」って要望があれば別ですけど。

[ds14050]

だいぶ理解してきました。

• CreateWindow で指定するコンボボックスの展開時高さはエディットボックス部分とリスト部分を合わせたもの
• CBS_NOINTEGRALHEIGHT を指定していないのでコンボボックスの展開時高さはリストアイテムに合わせて自動調整される
• コンボボックスのエディットボックス部分高さはフォントサイズにより自動調整される

CreateWindows の高さ指定が適当な手段ではなく、コンボボックスの垂直位置調整が WM_SETFONT よりあとでなければいけない理由です。

あっても邪魔はしないので残していますが、削ったほうがよければ削ります。

邪魔ではなくともデッドコードです。コメントと +1 を両方削除しておけば無駄に目を引くこともなく、+1 を +2 にしてみようという気を起こさせることもないでしょう。

「検索ボックスを1ドット下にずらした」という調整が「コンボボックスの垂直位置を調整する」によって発展的に置き換えられたと考えるなら消すのが当然と思います。

[berryzplus]

コメントと+1の件、ようやくコミット積めました。

[ds14050]

追加コミットありがとうございます。DPI 対応と同時にフォントサイズの変更にも対応できているのがいいですね。

[berryzplus]

レビューありがとうございます。