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

Grep置換ダイアログに誤追加された「否該当行」を削除 #1448

Merged

Conversation

suconbu
Copy link
Member

@suconbu suconbu commented Oct 31, 2020

PR の目的

#1431 の変更で誤って Grep 置換ダイアログに追加された「否該当行」ラジオボタンを削除します。

変更前 変更後
image image

カテゴリ

  • 不具合修正

PR の背景

PR のメリット

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

仕様・動作説明

日本語 / 英語リソースそれぞれの IDD_GREP_REPLACE から「否該当行」に相当するコントロールの定義を削除します。
近隣のアクセラレータキーの数字については Grep ダイアログとの共通性を持たせるため、削除される「3」以降を詰めることなくそのままとします。

テスト内容

日本語 / 英語それぞれで、Grep 置換ダイアログを表示させ「否該当行 (Not matched line)」がなくなっていることを確認します。

PR の影響範囲

ダイアログリソースの変更のみです。

関連 issue, PR

#1431 (不具合を入れ込んだ変更)

参考資料

@suconbu suconbu added the 🐛bug🦋 ■バグ修正(Something isn't working) label Oct 31, 2020
@AppVeyorBot
Copy link

Build sakura 1.0.3214 completed (commit 33988ea3c5 by @suconbu)

@berryzplus
Copy link
Contributor

強いて言えば、バグじゃないと思います。
だって、この変更はまだリリースしてないじゃんね?と言えてしまいますから。

@suconbu
Copy link
Member Author

suconbu commented Oct 31, 2020

Approve ありがとうございます。

@suconbu suconbu merged commit e54a074 into sakura-editor:master Oct 31, 2020
@suconbu suconbu removed the 🐛bug🦋 ■バグ修正(Something isn't working) label Oct 31, 2020
@beru beru added the 🐛bug🦋 ■バグ修正(Something isn't working) label Mar 21, 2021
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.

4 participants