emacs-helm / helm-dictionary

Helm source for looking up dictionaries
31 stars 12 forks source link

Doc: selected browser function #18

Closed michael-heerdegen closed 10 years ago

michael-heerdegen commented 10 years ago

Hi,

in the header and the doc of `helm-dictionary-browser-function', we should say more clearly how the browser function is selected. The docs are currently incomplete and inconsistent.

This is how it works:

Also the tag in

      (const      :tag "Currently configured default for Helm"
                     :value nil)

is misleading. We don't need to give a complete description in the :tag, we could just use "default" and explain in the docstring what that means.

tmalsburg commented 10 years ago

Fixed in 17f72225574dbba0ab8c128dea4ec00286f24e63. Thanks.