ルーラー上の数字フォントの幅が自動的に指定されるように LOGFONT 構造体の lfWidth メンバの値を 0 に変更 #505
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
berryzplus
approved these changes
Sep 30, 2018
| @@ -92,7 +92,7 @@ void CRuler::DrawRulerBg(CGraphics& gr) | |||
| HFONT hFontOld; | |||
| memset_raw( &lf, 0, sizeof(lf) ); | |||
| lf.lfHeight = 1 - pCommon->m_sWindow.m_nRulerHeight; // 2002/05/13 ai | |||
| lf.lfWidth = 5; | |||
| lf.lfWidth = 0; | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
変更前の固定値は…ルーラーの高さを標準設定値の 13 にした時にちょうど良い横幅って事なんでしょうか？
There was a problem hiding this comment.
変更前の挙動を見た感じ、
ルーラーの１目盛に収まるサイズを意識していたんではないかと思います。
標準フォントMS ゴシック 9ptの半角文字の幅は96dpiで6pxなので、
目盛線の太さ分1pxを引いて5pxということなのではないかと。
自分も変更しててそれ思いました。書体については気にしていませんでしたが、Ctrl 押しながらマウスホイール動かすとエディタ部分のフォントサイズ変えられますがルーラーの高さが変化しないんですよね。 設定画面での単位がドットなのでしょうがない気もしますが、単位でポイントも選べるようにして単位がポイントの場合は本文フォントサイズの変化に同期するのとかが良いんでしょうか？ |
|
Merge します。問題が見つかった場合は別の PR で修正する事にしましょう。 |
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 11, 2019
ルーラー上の数字フォントの幅が自動的に指定されるように LOGFONT 構造体の lfWidth メンバの値を 0 に変更
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
共通設定画面のウィンドウタブのルーラーの高さを大きくすると、ルーラー上の数字フォント（目盛り見出し？）の高さが大きくなりますが幅が広くならない為に不自然な表示になります。
LOGFONT 構造体の lfWidth メンバの値を 0 にする事で GDI 側で幅を自動的に計算してくれるのでそれを使うようにしました。
余談ですが、目盛り線の高さはルーラーの高さを変えても大きくなりません。しかしルーラーの高さに合わせて目盛り線の高さを調整すると数字とぶつかったりします（というかこのPRで横幅が広がると接触する）。個人的な意見としては、それぞれの要素の大きさを調整できるように設定項目を増やすのが良いと思います。
設定項目がむやみやたらに増えるとそれはそれで良くないので自動で良い塩梅に調整出来れば良いんですが、難しいです。