sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)
https://sakura-editor.github.io/
Other
1.24k stars 162 forks source link

タスクトレイから GREP置換ダイアログを出す機能が実装されていない #414

Open m-tmatma opened 5 years ago

m-tmatma commented 5 years ago

タスクトレイから GREP置換ダイアログを出す機能が実装されていない

grep ダイアログに関してはタスクトレイからメニューを出せる。 しかし、GREP置換ダイアログに関しては実装されていない。

https://github.com/sakura-editor/sakura/blob/4aa227c4aec130e1faa6a0f7da47b23fa0ca0e31/sakura_core/_main/CControlTray.cpp#L85-L90

https://github.com/sakura-editor/sakura/blob/4aa227c4aec130e1faa6a0f7da47b23fa0ca0e31/sakura_core/_main/CControlTray.h#L124

現状 grep ダイアログを出すのはコントロールプロセスなので #385 の対応がなくてもたいおう可能なはず。

berryzplus commented 5 years ago

需要があれば対応すべきだと思います。 ただ、実現方法は色々あって、タスクトレイよりいい方法があると思っています。

windows7以降、ユーザアプリによるタスクトレイの使用は非推奨になっているはずで、 代替手段(タスクバーアイコンからのジャンプリスト)が用意されています。 ジャンプリストからのコマンドをハンドルする処理はおそらく管理プロセスに実装することになりますが、外部要件が変わると実装方法も変わってくるのが自然です。

現状 grep ダイアログを出すのはコントロールプロセス

これについて、ぼくは現状の実装が正しくないと考えています。 問題というほど大きな問題ではないので issue には上げませんが、 管理プロセスにGrepダイアログを表示できる機能は必要ないと考えていて、 それを実現させるためだけにプログラム構造が複雑になっているのは良くないことだと思っています。

というわけで、機能導入には賛成、実装方法でCControlTrayにCGrepReplaceDialogのインスタンスを追加するのには反対です。やるなら #385 を対応して間接的に表示させられる仕組みを作る感じが良いと思っています。

m-tmatma commented 5 years ago

というわけで、機能導入には賛成、実装方法でCControlTrayにCGrepReplaceDialogのインスタンスを追加するのには反対です。やるなら #385 を対応して間接的に表示させられる仕組みを作る感じが良いと思っています。

それをするには事前に、grep ダイアログの場合の処理のリファクタリングをする必要があると思います。 grep ダイアログと grep 置換ダイアログで仕組みが違うとメンテが困難になります。

m-tmatma commented 5 years ago

410 で、同様に grep 置換ダイアログを出せるようにすれば、実質いらないかもしれませんが。

現状では grep はできるのに grep 置換はできないというのが不完全だと考えています。