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

質問:swiper対応について #1

Open
tam17aki opened this issue Oct 14, 2015 · 8 comments
Open

質問:swiper対応について #1

tam17aki opened this issue Oct 14, 2015 · 8 comments

Comments

@tam17aki
Copy link

便利なパッケージをありがとうございます。

READMEでご説明されているswiper対応についてですが、ivy--regex-migemoを定義してavy-migemo-modeをONにする以外には設定は必要ないのでしょうか?
例えば

(fset 'ivy--regex 'ivy--regex-migemo)

が必要だったりしますでしょうか?
よく分かっていなくて恐縮です。

@momomo5717
Copy link
Owner

ご使用いただきありがとうございます.

READMEでご説明されているswiper対応についてですが、ivy--regex-migemoを定義してavy-migemo-modeをONにする以外には設定は必要ないのでしょうか?

swiper対応の関数を avy-migemo-mode で使用する場合は,
(require 'avy-migemo-e.g.swiper) で動くと思います.

また,何かしらの関数を avy-migemo-mode に加える場合は,
新たに定義した関数(オリジナルの関数名に "-migemo" を加えた関数)を
(avy-migemo-add-names 'original-function-name-migemo)
のようにするだけで大丈夫だと思います.

フィードバックなどいただけたら幸いです.

@tam17aki
Copy link
Author

回答ありがとうございます。

swiper対応をした場合、動作確認(テスト)はどのようにしていますか?
たとえばswiper-helmの候補絞りこみ、とかでしょうか。

@momomo5717
Copy link
Owner

お返事ありがとうございます.

自分は swiper.el を使用しているので, swiper-helm.el では確認していませんでした.
今確認したところ,絞り込みはできているように思います.

ハイライトに関してはご容赦ください.自分の技量ではすぐに対応できそうにないです.

動作確認は ivy-test.el と同様なものを書くべきなのですが,
*scratch*バッファでテストをしたい文字列または,org-mode のファイルなどで確認しています.
また, ivy-recentf, ivy-switch-buffer を動かして確認しています.
入力に関しては,複数個の単語と簡単な正規表現(^$など)を混ぜたりして確認しています.
ただ,正規表現は結構失敗することがあります.

不十分なところが多々あると思いますので,
何か不具合などありましたら,お知らせいただけると幸いです.

@momomo5717
Copy link
Owner

すみません,swiper を更新したら3つ以上の単語を入れるとエラーになりました.

調べてみたところ,2015-10-07 のコミットで ivy-display-style の初期値に 'fancy が導入されたことが原因のようです.
ivy-display-style がどういう動きをするのかまだ調べてないのですが,
(setq ivy-display-style nil)
で動くと思います.

お手数をおかけして申し訳ありません.

@tam17aki
Copy link
Author

swiper対応はとりあえずうまく動いているように思います。ありがとうございます。
しばらく使ってみたいと思います。

私はEmacs24.4を用いていたので,ご指摘の問題点は幸いにも回避できていたようです。
Emacs24.5に更新して動作検証する気力はないので笑、対応はお任せします。

@momomo5717
Copy link
Owner

動作したということで安心しました.先程更新した瞬間かなり焦りました:sweat_smile:.

ご使用中何かありましたら,お知らせいただけると幸いです.

お礼を申し上げるのが遅くなりましたが, helm-migemo-mode の件ありがとうございました!
さらに,いつの間にか avy-migemo の記事が!ありがとうございます.
今後ともよろしくお願いいたします。

momomo5717 added a commit that referenced this issue Oct 15, 2015
Add avy-migemo--backup-ivy-display-style

Error occured when three or more patterns are inputed in ivy-display-style's fancy.
So ivy-display-style is set to nil, when avy-migemo-mode is enabled.
momomo5717 added a commit that referenced this issue Oct 18, 2015
Remove avy-migemo--backup-ivy-display-style
momomo5717 added a commit that referenced this issue Oct 18, 2015
@tam17aki
Copy link
Author

更新お疲れ様です。

ところで、avy-migemo-use-isearch-search-fun のどのような理由で導入したのでしょうか?
全然分かっていなくてすみません。

@momomo5717
Copy link
Owner

ご質問ありがとうございます.

ところで、avy-migemo-use-isearch-search-fun のどのような理由で導入したのでしょうか?

できるだけ isearch と同じような選択候補を表示したい,という理由で追加しました.

特に migemo-isearch-toggle-migemo で migemo をオフにした場合でも migemo による候補が含まれてしまうためです.また word search や symbol search から avy を呼び出す場合も同様に意図しない候補が入ってしまうためです.

ただ migemo-do-isearchno-nil の場合に,migemo-backward が他の search系関数のように動作しなかったので,migemo-backward と同じような動きをする関数が isearch-search-fun-function で設定されている場合はavy-migemo--isearch-candidates が動かないかもしれません.
そのときは新たに対策を考えようと思います.

お役に立てれば幸いです.

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

No branches or pull requests

2 participants