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.

Sign up for GitHub

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

Jump to bottom

Grep/Grep置換ダイアログの余白調整/文言統一 #1672

Merged
merged 3 commits into from
May 21, 2021
Merged

Grep/Grep置換ダイアログの余白調整/文言統一 #1672

merged 3 commits into from
May 21, 2021

Conversation

suconbu
Copy link
Member

@suconbu suconbu commented May 21, 2021 •
edited
Loading

PR の目的

Grep/Grep置換ダイアログについて、

  1. 余白を調整することで見た目を良くしつつ、ダイアログの高さをほんの少しだけ (1～3%) コンパクトにします。
  2. 「単語単位」「英大文字小文字区別」の文言を検索ダイアログと同じもの/ヘルプ記載のものに戻します。

#1431 の対応で狙っていた通りの配置になっていない点を修正しつつ、
#1622 で挙がっていたGrep/Grep置換ダイアログに関する要望を (部分的ですが) 対応します。

Grep/Grep置換ダイアログの見た目比較：
(ディスプレイ設定の表示スケールは 100%、システムフォント/フォント設定のフォントは共に「Yu Gothic UI」)

PR 反映前 PR 反映後
Grep (日本語) image image
Grep置換 (日本語) image image
Grep (英語) image image
Grep置換 (英語) image image

カテゴリ

  • 不具合修正 (以前の対応の不備を修正)

PR の背景

目的に記載の通りです。

PR のメリット

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

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

特にないと思います。

仕様・動作説明

1. 余白の調整

「Grep/Grep置換ダイアログの見た目比較」の赤枠で示した部分の余白を、#1431 の参考資料に掲載の配置ルールに基づき 4DLU になるようにします。
これに合わせてダイアログボックス全体の高さも調整します。

2. 文言の修正

変更前 変更後
単語単位(W) 単語単位で探す(W)
英大小文字区別(C) 英大文字と小文字を区別する(C)

#1431 で文言を変更していた背景としては、
作業当初にチェックボックスを横並びで検討していた際、幅に収まるよう文言を短くしていたが、その後チェックボックスを縦並びに見直した後もそのままとしていました。

PR の影響範囲

Grep/Grep置換ダイアログ (日本語/英語) の見た目が変化します。

テスト内容

Grep/Grep置換ダイアログで、文字切れや文字列/コントロールの重なりがないことを確認します。

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

関連 issue, PR

#1622
#1431
#1421

参考資料

@sonarqubecloud
Copy link

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

suconbu
suconbu commented May 21, 2021
View reviewed changes
sakura_core/sakura_rc.rc
@@ -3409,8 +3414,6 @@ BEGIN
STR_ERR_DLGEDITVW6 "■"
STR_MENU_KEYWORDINFO "キーワードの説明をクリップボードにコピー"
STR_MENU_OPENKEYWORDDIC "キーワード辞書を開く"
STR_STATUS_FONTZOOM_0 "%4.0f %%"
Copy link
Member Author

@suconbu suconbu May 21, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

以下、今回の変更と関係のない差分がありますが、これはリソースを VisualStudio から保存した時に出てしまったものです。
最初、この関係のない部分をコミットから除外しようと思いましたが、次回このファイルを修正する際にまた同じことをしなければならなくなるはずなので、できればこのままとさせてもらいたいです。

@suconbu suconbu marked this pull request as ready for review May 21, 2021 08:43
@AppVeyorBot
Copy link

✅ Build sakura 1.0.3779 completed (commit b4c94e7f4c by @suconbu)

@AppVeyorBot
Copy link

✅ Build sakura 1.0.3780 completed (commit 043cc415b4 by @suconbu)

sanomari
sanomari approved these changes May 21, 2021
View reviewed changes
Copy link
Contributor

@sanomari sanomari left a comment

Choose a reason for hiding this comment

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

レイアウト変更は妥当に見えました。

リソーススクリプトが意図せず変更された件は不可解ですが、デザイナで触って変更されたのなら戻さなくていいと思います。

@suconbu
Copy link
Member Author

suconbu commented May 21, 2021

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

@suconbu suconbu merged commit 0843eb9 into sakura-editor:master May 21, 2021
@beru beru mentioned this pull request May 23, 2021
Closed
@beru beru added the 🐛bug🦋 ■バグ修正(Something isn't working) label Oct 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanomari sanomari sanomari approved these changes

Assignees
No one assigned
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@suconbu @AppVeyorBot @sanomari @beru