dash-docs-el / helm-dash

Browse Dash docsets inside emacs
511 stars 59 forks source link

helm by default #55

Closed kidd closed 10 years ago

kidd commented 10 years ago

related to #51

I don't like that by default helm allows not matching results.... so as it is now I'd be against merging it. wdyt?

areina commented 10 years ago

Well, you can pass :must-match t to helm-comp-read. The problem then is that probably those args are not compatible with ido-completing-read. We should think a nice way to do that. Remember that we use helm-comp-read on helm-dash-activate-docset and helm-dash-install-docset

Another detail, you should add helm-comp-read to :options in helm-dash-completing-read-func, and

kidd commented 10 years ago

@areina wdyt?

areina commented 10 years ago

It's perfect form me!