Re: [O] [PATCH] Add a custom list in org-mac-link.el

2017-06-15 Thread Nicolas Goaziou
Takaaki Ishikawa writes: > Thank you for your kind feedback. Please find my comments below. Thank you for the quick answer. > Updated docstring is > > "Alist of descriptors. Each descriptor consists of four elements to build > the link grabber menu in the minibuffer. A

Re: [O] [PATCH] Add a custom list in org-mac-link.el

2017-06-15 Thread Takaaki Ishikawa
Dear Nicolas, Thank you for your kind feedback. Please find my comments below. > +(defcustom org-mac-link-descriptors > > + `(("F" "inder" org-mac-finder-insert-selected > ,org-mac-grab-Finder-app-p) > > +("m" "ail" org-mac-message-insert-selected > ,org-mac-grab-Mail-app-p) > > +("d"

Re: [O] [PATCH] Add a custom list in org-mac-link.el

2017-06-15 Thread Nicolas Goaziou
Hello, Takaaki Ishikawa writes: > I tried to generate a patch for this proposal. Please find an attached > file. Thank you. > +(defcustom org-mac-link-descriptors > + `(("F" "inder" org-mac-finder-insert-selected ,org-mac-grab-Finder-app-p) > +("m" "ail"

Re: [O] [PATCH] Add a custom list in org-mac-link.el

2017-06-15 Thread Takaaki Ishikawa
Dear Nicolas and all, I tried to generate a patch for this proposal. Please find an attached file. If you have any comments or suggestions on the file, feel free to ask me. Best, Takaaki Takaaki ISHIKAWA GITI, Waseda University :) http://about.me/takaxp 2017-06-15 1:37

Re: [O] [PATCH] Add a custom list in org-mac-link.el

2017-06-14 Thread Nicolas Goaziou
Hello, Takaaki Ishikawa writes: > The attached patch includes a change for adding a custom variable to easily > extend link grabbing capabilities for masOS user. When someone wants to add > an additional menu to call an external application, it will be possible by > adding a

[O] [PATCH] Add a custom list in org-mac-link.el

2017-06-11 Thread Takaaki Ishikawa
Dear folks, The attached patch includes a change for adding a custom variable to easily extend link grabbing capabilities for masOS user. When someone wants to add an additional menu to call an external application, it will be possible by adding a setting to the proposed variable as follows: