GREPダイアログ内の全コンボボックスに対してフォント設定で指定されたフォントを使用する

[suconbu]

PR の目的

Grep ダイアログ・Grep 置換ダイアログのコンボボックスの視認性を改善します。

変更前：

変更後：

カテゴリ

• 仕様変更

PR の背景

Grep ダイアログの除外ファイルなどに表示されているピリオドやセミコロンなどの文字が小さく見づらく、入力間違いをしていても気づけないことが時々あるため。

PR のメリット

コンボボックス上の文字が少しだけ見やすくなります。

PR のデメリット (トレードオフとかあれば)

コンボボックスに入力される文字数が多い時に従来よりはみ出しやすくなります。

仕様・動作説明

• Grep ダイアログ・GREP 置換ダイアログのコンボボックス※で使用するフォントを、条件テキストボックスと同じ等幅フォントで統一します。
  ※ファイル・フォルダ・除外ファイル・除外フォルダが変更の対象です。

テスト内容

テスト1

1. GREP ダイアログを表示する。

• 確認1: ファイル・フォルダ・除外ファイル・除外フォルダのコンボボックスのフォントがそれぞれ等幅になっていること

同様のテストを GREP 置換ダイアログでも実施する。

PR の影響範囲

Grep ダイアログ・Grep 置換ダイアログ

関連 issue, PR

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.3085 completed (commit 3122739372 by @suconbu)

[k-takata]

sakura_core/sakura_rc.aps が追加されていますが、削除してください。
コミットを編集して force push するのがよいでしょう。

[ghost]

Grepダイアログの条件欄以外のコンボボックスは、今までフォント設定で指定したフォントが反映されないようでした。
（条件欄とGrepじゃないほうの検索・置換ダイアログは反映される）
本PRでは新たに反映されるようになり嬉しいです。

※フォント設定で和文等幅フォントの「Migu 1M」を選択している場合のダイアログ
（円記号がバックスラッシュになっていますが、これは正常な表示です）

一方で、フォント設定ではプロポーショナル（可変幅）フォントが選択できるため、
設定次第ではありますが、当該コンボボックスにはプロポーショナルフォントが表示される可能性があります。

[suconbu]

k-takata さん、beru さん、レビューして頂きありがとうございます。
指摘頂いた点を修正しましたのでご確認お願いします。

[AppVeyorBot]

✅ Build sakura 1.0.3087 completed (commit 4ddb551977 by @suconbu)

[suconbu]

一方で、フォント設定ではプロポーショナル（可変幅）フォントが選択できるため、
設定次第ではありますが、当該コンボボックスにはプロポーショナルフォントが表示される可能性があります。

kazasaku さんにコメント頂いた通り、必ずしも等幅で表示される訳ではなかったため、
PR の題名や説明中の文言は適当でなかったかもしれません。もし修正必要であればご指摘お願いします。

[beru]

Grepダイアログの条件欄以外のコンボボックスは、今までフォント設定で指定したフォントが反映されないようでした。
（条件欄とGrepじゃないほうの検索・置換ダイアログは反映される）
本PRでは新たに反映されるようになり嬉しいです。

フォント設定で指定したフォントが検索系のダイアログのテキストボックス（とかコンボボックスとかのコントロール）のフォントにも反映されるの知りませんでした。

こういう風に Webdings フォントを常用している人にも優しい変更ですね。ただアウトライン解析の方はフォントが反映されないので Webdings フォントを設定していても通常のフォントが設定されたままで、関数名をクリックしてフォーカスを移して表示を見ても何が何だか良く分かりません。アウトライン解析の方にも Webdings フォントが設定されればそちらの表示も何が何だか良く分からなくなるのでより対応が取れるような気がします。

[beru]

問題無いと思います。

[beru]

kazasaku さんにコメント頂いた通り、必ずしも等幅で表示される訳ではなかったため、
PR の題名や説明中の文言は適当でなかったかもしれません。もし修正必要であればご指摘お願いします。

仮に実態に合ったコメントにするとしたらどういう文面が良いんですかね？

「GREPダイアログのコントロールのフォント設定を徹底する」とかですか？

[ghost]

suconbu様、beru様、ご確認ありがとうございます。

自分がMSゴシックが嫌いな人間でMigu 1Mを使っていたので、たまたま試していて気が付いた次第です。
デフォルトのフォントだと気が付かなかったかもしれません。
気に留めていただけたようで何よりです。

[suconbu]

仮に実態に合ったコメントにするとしたらどういう文面が良いんですかね？

なかなか短くまとめるのは難しいですが、今回の変更内容を表現しようとすると、
「GREPダイアログ内の全コンボボックスに対してフォント設定で指定されたフォントを使用する」
などでしょうか。

[beru]

なかなか短くまとめるのは難しいですが、今回の変更内容を表現しようとすると、
「GREPダイアログ内の全コンボボックスに対してフォント設定で指定されたフォントを使用する」
などでしょうか。

分かりやすくて良いと思います。

[berryzplus]

見やすくなる変更なのでよいと思います。

不揃いになってるのを同じにするんだから、統一する？

このPRではフォントを揃えてるだけなので「等幅フォント化」ってのは違うような気もします。

[ghost]

仮に実態に合ったコメントにするとしたらどういう文面が良いんですかね？

なかなか短くまとめるのは難しいですが、今回の変更内容を表現しようとすると、
「GREPダイアログ内の全コンボボックスに対してフォント設定で指定されたフォントを使用する」
などでしょうか。

自分もこの内容でよいと思います。
（改めて見返して、自分のコメントが分かりにくいと感じたので再度意思表示しておきます。）

[suconbu]

レビュー・マージありがとうございました。
仕事で趣味で長くお世話になっているサクラエディタに貢献ができうれしいです😃
申し遅れましたが suconbu と申します。今後ともよろしくお願いします。