From: Tomi Ollila Date: Mon, 20 Oct 2014 18:53:14 +0000 (+0300) Subject: Re: [PATCH] VIM: Add URI handling X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=1ee25c274212e2f174b8069a2c9aee09ff7cf2fd;p=notmuch-archives.git Re: [PATCH] VIM: Add URI handling --- diff --git a/29/8ac074c4138d22124640012b0cc5c2aba5dae7 b/29/8ac074c4138d22124640012b0cc5c2aba5dae7 new file mode 100644 index 000000000..28977b796 --- /dev/null +++ b/29/8ac074c4138d22124640012b0cc5c2aba5dae7 @@ -0,0 +1,217 @@ +Return-Path: +X-Original-To: notmuch@notmuchmail.org +Delivered-To: notmuch@notmuchmail.org +Received: from localhost (localhost [127.0.0.1]) + by olra.theworths.org (Postfix) with ESMTP id 216F2431FB6 + for ; Mon, 20 Oct 2014 11:53:41 -0700 (PDT) +X-Virus-Scanned: Debian amavisd-new at olra.theworths.org +X-Spam-Flag: NO +X-Spam-Score: 0 +X-Spam-Level: +X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none] + autolearn=disabled +Received: from olra.theworths.org ([127.0.0.1]) + by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) + with ESMTP id UyJG36pqWdGg for ; + Mon, 20 Oct 2014 11:53:37 -0700 (PDT) +Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34]) + by olra.theworths.org (Postfix) with ESMTP id 8182D431FAF + for ; Mon, 20 Oct 2014 11:53:37 -0700 (PDT) +Received: from guru.guru-group.fi (localhost [IPv6:::1]) + by guru.guru-group.fi (Postfix) with ESMTP id BD66D10008C; + Mon, 20 Oct 2014 21:53:14 +0300 (EEST) +From: Tomi Ollila +To: Ian Main , notmuch@notmuchmail.org +Subject: Re: [PATCH] VIM: Add URI handling +In-Reply-To: <1412281423-22441-1-git-send-email-imain@stemwinder.org> +References: <1412281423-22441-1-git-send-email-imain@stemwinder.org> +User-Agent: Notmuch/0.18.1+130~ga61922f (http://notmuchmail.org) Emacs/24.3.1 + (x86_64-unknown-linux-gnu) +X-Face: HhBM'cA~ +MIME-Version: 1.0 +Content-Type: text/plain +X-BeenThere: notmuch@notmuchmail.org +X-Mailman-Version: 2.1.13 +Precedence: list +List-Id: "Use and development of the notmuch mail system." + +List-Unsubscribe: , + +List-Archive: +List-Post: +List-Help: +List-Subscribe: , + +X-List-Received-Date: Mon, 20 Oct 2014 18:53:41 -0000 + +On Thu, Oct 02 2014, Ian Main wrote: + +> This patch adds URI handling to the vim client. You can now press +> 'u' by default and the client will parse the current line and find +> any URIs available. If there are more than one it opens the one +> under the cursor or else it opens the only one available. It also +> supports mailto: URI's and will compose a new message when activated. +> +> By default xdg-open is used for everything but mailto: which generally +> does the right thing afaict. + +This patch LGTM. + +> +> Ian + +Tomi + +> --- +> vim/notmuch.txt | 1 + +> vim/notmuch.vim | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++------- +> 2 files changed, 53 insertions(+), 7 deletions(-) +> +> diff --git a/vim/notmuch.txt b/vim/notmuch.txt +> index 4374102..d336406 100644 +> --- a/vim/notmuch.txt +> +++ b/vim/notmuch.txt +> @@ -72,6 +72,7 @@ q Quit view +> A Archive (-inbox -unread) +> I Mark as read (-unread) +> t Tag (prompted) +> +u Open URI +> s Search +> p Save patches +> r Reply +> diff --git a/vim/notmuch.vim b/vim/notmuch.vim +> index 331e930..de82bb9 100644 +> --- a/vim/notmuch.vim +> +++ b/vim/notmuch.vim +> @@ -12,7 +12,7 @@ let g:notmuch_folders_maps = { +> \ '': 'folders_show_search()', +> \ 's': 'folders_search_prompt()', +> \ '=': 'folders_refresh()', +> - \ 'c': 'compose()', +> + \ 'c': 'compose("")', +> \ } +> +> let g:notmuch_search_maps = { +> @@ -25,7 +25,7 @@ let g:notmuch_search_maps = { +> \ 's': 'search_search_prompt()', +> \ '=': 'search_refresh()', +> \ '?': 'search_info()', +> - \ 'c': 'compose()', +> + \ 'c': 'compose("")', +> \ } +> +> let g:notmuch_show_maps = { +> @@ -37,10 +37,11 @@ let g:notmuch_show_maps = { +> \ 'e': 'show_extract_msg()', +> \ 's': 'show_save_msg()', +> \ 'p': 'show_save_patches()', +> + \ 'u': 'show_open_uri()', +> \ 'r': 'show_reply()', +> \ '?': 'show_info()', +> \ '': 'show_next_msg()', +> - \ 'c': 'compose()', +> + \ 'c': 'compose("")', +> \ } +> +> let g:notmuch_compose_maps = { +> @@ -59,6 +60,7 @@ let s:notmuch_datetime_format_default = '%d.%m.%y %H:%M:%S' +> let s:notmuch_reader_default = 'mutt -f %s' +> let s:notmuch_sendmail_default = 'sendmail' +> let s:notmuch_folders_count_threads_default = 0 +> +let s:notmuch_open_uri_default = 'xdg-open' +> +> function! s:new_file_buffer(type, fname) +> exec printf('edit %s', a:fname) +> @@ -135,8 +137,8 @@ function! s:show_reply() +> startinsert! +> endfunction +> +> -function! s:compose() +> - ruby open_compose +> +function! s:compose(to_email) +> + ruby open_compose(VIM::evaluate('a:to_email')) +> let b:compose_done = 0 +> call s:set_map(g:notmuch_compose_maps) +> autocmd BufDelete call s:on_compose_delete() +> @@ -159,6 +161,45 @@ ruby << EOF +> EOF +> endfunction +> +> +function! s:show_open_uri() +> + let line = getline(".") +> + let pos = getpos(".") +> + let col = pos[2] +> +ruby << EOF +> + m = get_message +> + line = VIM::evaluate('line') +> + col = VIM::evaluate('col') - 1 +> + uris = URI.extract(line) +> + wanted_uri = nil +> + if uris.length == 1 +> + wanted_uri = uris[0] +> + else +> + uris.each do |uri| +> + # Check to see the URI is at the present cursor location +> + idx = line.index(uri) +> + if col >= idx and col <= idx + uri.length +> + wanted_uri = uri +> + break +> + end +> + end +> + end +> + +> + if wanted_uri +> + uri = URI.parse(wanted_uri) +> + if uri.class == URI::MailTo +> + vim_puts("Composing new email to #{uri.to}.") +> + VIM::command("call s:compose('#{uri.to}')") +> + else +> + vim_puts("Opening #{uri.to_s}.") +> + cmd = VIM::evaluate('g:notmuch_open_uri') +> + system(cmd, uri.to_s) +> + end +> + else +> + vim_puts('URI not found.') +> + end +> +EOF +> +endfunction +> + +> function! s:show_open_msg() +> ruby << EOF +> m = get_message +> @@ -404,6 +445,10 @@ function! s:set_defaults() +> endif +> endif +> +> + if !exists('g:notmuch_open_uri') +> + let g:notmuch_open_uri = s:notmuch_open_uri_default +> + endif +> + +> if !exists('g:notmuch_reader') +> if exists('g:notmuch_rb_reader') +> let g:notmuch_reader = g:notmuch_rb_reader +> @@ -611,11 +656,11 @@ ruby << EOF +> open_compose_helper(lines, cur) +> end +> +> - def open_compose() +> + def open_compose(to_email) +> lines = [] +> +> lines << "From: #{$email}" +> - lines << "To: " +> + lines << "To: #{to_email}" +> cur = lines.count +> +> lines << "Cc: " +> -- +> 1.9.3 +> +> _______________________________________________ +> notmuch mailing list +> notmuch@notmuchmail.org +> http://notmuchmail.org/mailman/listinfo/notmuch