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置換ダイアログの「置換後」の実装を置換ダイアログと合わせる #1224

Conversation

berryzplus
Copy link
Contributor

PR の目的

Grep置換ダイアログの「置換後」の実装を置換ダイアログと合わせます。

カテゴリ

  • 仕様変更 (Grep置換後ダイアログの置換後ボックスで履歴削除ができるようになります。)
  • 不具合修正 (履歴機能付きコンボの共通仕様が実装されていない、という不具合を修正します。)

PR の背景

#1219 を参照してください。

PR のメリット

  • 明らかな実装漏れを、とりあえず補うことができます。
  • 動いている既存コードのコピペなので、安心して使うことができます。
  • Grep置換ダイアログの置換後ボックスでDeleteキーによる履歴削除を利用できるようになります。

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

  • 置換ダイアログの実装のコピペです。
    • 置換ダイアログの既存実装を見直す貴重な機会をスルーする変更です。
    • Grep置換ダイアログの実装にそのまま加えて問題ないかの検証をスルーする変更です。

PR の影響範囲

  • アプリ(=サクラエディタ)の挙動に影響がある変更です。 ◦検索ダイアログの検索ボックス
    • Grep置換ダイアログの置換後ボックス
      • ドロップダウン中にDeleteキーを押下すると選択した履歴を消せるようになります。

関連チケット

#1223
#1221
#1219

参考資料

@AppVeyorBot
Copy link

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.

動作確認をしました。問題無いと思います。
Grep置換ダイアログの置換後のリストアイテムがDelキーで削除出来なかったのが出来るようになりました。

@m-tmatma m-tmatma added this to the next release (after v2.4.0) milestone Apr 19, 2020
@berryzplus
Copy link
Contributor Author

レビューありがとうございます。
マージしちゃいます。

@berryzplus berryzplus merged commit 86311fe into sakura-editor:master Apr 21, 2020
@berryzplus berryzplus deleted the feature/add_recent-rubber_to_grepreplace-replace branch April 21, 2020 00:35
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…ent-rubber_to_grepreplace-replace

Grep置換ダイアログの「置換後」の実装を置換ダイアログと合わせる
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.

4 participants