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.

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

印刷ページ設定ダイアログの文字切れ等の修正 #1674

Merged

Conversation

suconbu
Copy link
Member

@suconbu suconbu commented May 21, 2021

PR の目的

#1622 で印刷ページ設定ダイアログの文字切れの報告がありました。
この PR ではその修正と、追加で見つけた文字切れ/重なりを合わせて直します。

  1. Grep(置換)と印刷ページ設定のダイアログをWindows8.1で崩れないように変更 #1622 で報告のあった文字切れ (システムフォントが「Meiryo UI」の時に発生)
修正前 修正後
image image
  1. 「ヘッダー」「フッター」の末尾の文字切れ (システムフォントが「メイリオ」の時に発生)
修正前 修正後
image image
  1. 英語モードでの文字切れ/コントロール同士の重なり (これは「Yu Gothic UI」でも発生)
修正前 修正後
image image

カテゴリ

  • 不具合修正

PR の背景

目的に記載の通りです。

PR のメリット

こちらも目的に記載の通りです。

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

特にないと思います。

仕様・動作説明

「1」については #1622 に上げて頂いているコミットを適用し修正、
「2」「3」については sakura_rc.rc を直接編集して調整を行います。

なお、調整後でも下図のようにヘッダー/フッター用として選択したフォント名によっては文字切れが発生する場合があります。
これはもうダイアログの配置を全体的に見直すことなしに解消することは難しいので、今回は対策を見送ることにします。

image

PR の影響範囲

印刷ページ設定ダイアログ (日本語/英語) の見た目が変化します。

テスト内容

印刷ページ設定ダイアログで、文字切れや文字列/コントロールの重なりがないことを確認します。

言語:日本語/英語
システムフォント:「Yu Gothic UI」/「Meiryo UI」/「メイリオ」

関連 issue, PR

#1622
#1421

参考資料

@sonarcloud
Copy link

sonarcloud bot commented May 21, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@AppVeyorBot
Copy link

Build sakura 1.0.3783 completed (commit 800a7822a1 by @suconbu)

Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

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

動作確認は行っていませんが、変更は妥当に見えます。

印刷設定ダイアログのレイアウトについては、そもそもなツッコミどころが1つあって、それを考慮すると「設定」グループボックス(STATICコントロール)の位置・サイズが微妙に変わるんですが見なかったことにします。

@berryzplus
Copy link
Contributor

berryzplus commented May 22, 2021

書くだけ書いておきます。

👇は印刷設定ダイアログのデザイナ表示です。
image

DelphiやVisual BasicなどのRAD開発環境を使ったことある人なら、
このデザイナ画面の「不審点」が分かるかも知れません。

RADツールでは、画面レイアウトの「外枠」を定義する仕組みがあります。
Excelの印刷プレビューにある「印刷範囲」と近い概念のものです。

上の画像で、OKボタンの下端近くにある薄い線が外枠線です。
見たらわかる通り、印刷設定ダイアログでは、下端も左端も右端も外枠が無視されています。
この画面は設計的に、左端を無視したグループボックスを配置する構成なので、左端の飛び出しは無視してよいと思います。しかし、下端と右端のはみ出しはNGっぽいです。

直さないといけないのか? というとそうでもないっす。

なんでこうなってるんだっけ? と考えると、
「初期作成者がVisual Studioを利用していなかった」が混入原因で、
「メンテナがVisual Studioの使い方を理解してなかった」が放置されてきた原因になる気がします。

デザイナの使い方は、Visual Studioの使い方の中では「初心者レベル」に該当する話題です。

以上のことを踏まえて、放置しといていいんだっけ?というのが若干気になったのですが、PRの趣旨とは関係ないのでスルーした次第です。

こういうしょうもないことに対応できないことが、仲間と思われたくないに繋がって、メンバー募集しても誰も入ってくれない結果に繋がっていってるんじゃないかとか思ったりしています。。。

@suconbu
Copy link
Member Author

suconbu commented May 22, 2021

ありがとうございます。マージしてしまいます。

@suconbu suconbu merged commit a9857e2 into sakura-editor:master May 22, 2021
@suconbu
Copy link
Member Author

suconbu commented May 22, 2021

リソースファイルに関しては、#1456 で VisualStudio から開く/保存するができるようにはなったものの、大半のダイアログリソースはまだデザイナを使って編集できる状態ではないですね。
コメント頂いた外枠線の位置もめちゃめちゃですし、そもそもリソースに指定されたフォント (MS Pゴシック) と実際に表示されるフォント (Yu Gothic UI) とが違っているため※にデザイナ上での位置合わせができず、テキストエディタでの修正を強いられます。
#1421 の対応でそうしてしまったのは私ですが😓

数が数 (現時点で 61 (x2) 箱あるようです) なので一気には難しいですが、ちょっとずつでも直して※いきたい気持ちはあります。
※想定される作業としては「リソースのフォント指定を Yu Gothic UI に設定」した後「フォント変更によって間延びした配置を元のサイズ感を参考に『いい感じ』に調整」です。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants