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

「フォルダー」のカタカナ表記をOSに合わせる #1841

Merged
merged 2 commits into from May 6, 2022

Conversation

sanomari
Copy link
Contributor

@sanomari sanomari commented May 5, 2022

PR の目的

タイトル通りです。

カテゴリ

  • 仕様変更
  • リファクタリング
  • ドキュメント修正

PR の背景

#1822 でサクラエディタ内の「フォルダ」がOSの表記「フォルダー」と異なっていることが指摘されました。

PR のメリット

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

仕様・動作説明

  • 「フォルダ」を「フォルダー」に変更します。
  • 変更によりラベルに「フォルダー」を含むダイアログコントロールの一部で文字が見切れたため、サイズ変更と周辺コントロールの位置調整を行います。

PR の影響範囲

  • コメントとドキュメント、インストーラーに影響する変更です。

テスト内容

修正が必要なダイアログが複数あり、しかも表示確認しづらいもの(Grep実行中ダイアログ)が含まれるので、テスト内容はいったん割愛します。

テスト1

手順

関連 issue, PR

#1822

参考資料

@sonarcloud
Copy link

sonarcloud bot commented May 5, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

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

32.7% 32.7% Coverage
1.3% 1.3% Duplication

@AppVeyorBot
Copy link

Build sakura 1.0.4125 completed (commit 7416ba22e2 by @sanomari)

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.

対象ファイルが非常に多いので「時間をかけるのはマズそう」と思っています。

とりあえず変更は「フォルダ」を「フォルダー」にする置換とダイアログの位置調整以外にはなさそうでした。リソースエディタで開いて見た限り、位置調整は妥当に見えました。

Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

ざっと確認した限り問題無さそうです。

@sanomari sanomari merged commit 7c5b17d into sakura-editor:master May 6, 2022
@sanomari sanomari deleted the feature/fix_text_of_folder branch May 6, 2022 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants