From: Tomi Ollila <tomi.ollila@iki.fi>
To: Ian Main <imain@stemwinder.org>, notmuch@notmuchmail.org
Subject: Re: [PATCH v3] VIM: Add URI handling
Date: Fri, 24 Oct 2014 19:30:29 +0300 [thread overview]
Message-ID: <m2oat177mi.fsf@guru.guru-group.fi> (raw)
In-Reply-To: <1414167094-31566-1-git-send-email-imain@stemwinder.org>
On Fri, Oct 24 2014, Ian Main <imain@stemwinder.org> wrote:
This patch does not apply on top of notmuchmail master
(commit 38240d106139da8).
> This patch adds URI handling to the vim client. You can now press
Although insignificant, I'll start commenting on all sent patches
(that I look into) which talk like 'This patch adds' -- sure it is patch
when email is sent, but in repository it is not so -- so the commit message
should not mention it (but I do not require changing it).
Anyway, I suspect this change will become applicable after some other
change is committed first -- and IMO this could stay as non-stale patch
provided that SomeOne(TM) informs what is the message that contains
changes (or series those) that is required for this message to apply.
BTW: does the content of this change differ much from v1 at
id:1412281423-22441-1-git-send-email-imain@stemwinder.org
or should I re-check (carefully!) that the changes are still OK.
Tomi
> 'enter' by default and the client will parse the current line and find
> any 'Part's or URIs available for opening. 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.
>
> Note that this is now dependant on the attachment patch in order to
> make the nice 'enter' behavior work for both.
>
> Ian
> ---
>
> Fix commit message formatting.
>
> vim/notmuch.txt | 3 ++-
> vim/notmuch.vim | 76 +++++++++++++++++++++++++++++++++++++++++++++++++++------
> 2 files changed, 70 insertions(+), 9 deletions(-)
>
> diff --git a/vim/notmuch.txt b/vim/notmuch.txt
> index 838a904..5d84fde 100644
> --- a/vim/notmuch.txt
> +++ b/vim/notmuch.txt
> @@ -74,7 +74,8 @@ I Mark as read (-unread)
> t Tag (prompted)
> e Extract attachment on the current 'Attachment' line or all
> attachments if the cursor is elsewhere.
> -v View attachment on the current 'Attachment' line.
> +<enter> View email part on the current 'Part' line, or open URI under cursor
> + or on line.
> s Search
> p Save patches
> r Reply
> diff --git a/vim/notmuch.vim b/vim/notmuch.vim
> index 1466e50..2f76f55 100644
> --- a/vim/notmuch.vim
> +++ b/vim/notmuch.vim
> @@ -12,7 +12,7 @@ let g:notmuch_folders_maps = {
> \ '<Enter>': '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 = {
> @@ -35,13 +35,13 @@ let g:notmuch_show_maps = {
> \ 't': 'show_tag("")',
> \ 'o': 'show_open_msg()',
> \ 'e': 'show_extract_msg()',
> - \ '<Enter>': 'show_view_attachment()',
> + \ '<Enter>': 'show_view_magic()',
> \ 's': 'show_save_msg()',
> \ 'p': 'show_save_patches()',
> \ 'r': 'show_reply()',
> \ '?': 'show_info()',
> \ '<Tab>': 'show_next_msg()',
> - \ 'c': 'compose()',
> + \ 'c': 'compose("")',
> \ }
>
> let g:notmuch_compose_maps = {
> @@ -63,6 +63,7 @@ let s:notmuch_view_attachment_default = 'xdg-open'
> let s:notmuch_attachment_tmpdir_default = '~/.notmuch/tmp'
> let s:notmuch_folders_count_threads_default = 0
> let s:notmuch_compose_start_insert_default = 1
> +let s:notmuch_open_uri_default = 'xdg-open'
>
> function! s:new_file_buffer(type, fname)
> exec printf('edit %s', a:fname)
> @@ -141,8 +142,8 @@ function! s:show_reply()
> end
> 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 <buffer> call s:on_compose_delete()
> @@ -155,6 +156,22 @@ function! s:show_info()
> ruby vim_puts get_message.inspect
> endfunction
>
> +function! s:show_view_magic()
> + let line = getline(".")
> +
> +ruby << EOF
> + line = VIM::evaluate('line')
> +
> + # Easiest to check for 'Part' types first..
> + match = line.match(/^Part (\d*):/)
> + if match and match.length == 2
> + VIM::command('call s:show_view_attachment()')
> + else
> + VIM::command('call s:show_open_uri()')
> + end
> +EOF
> +endfunction
> +
> function! s:show_view_attachment()
> let line = getline(".")
> ruby << EOF
> @@ -226,6 +243,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
> @@ -481,6 +537,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
> @@ -693,11 +753,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
next prev parent reply other threads:[~2014-10-24 16:31 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-02 20:23 [PATCH] VIM: Add URI handling Ian Main
2014-10-10 9:44 ` Franz Fellner
2014-10-10 18:18 ` Ian Main
2014-10-11 12:29 ` Franz Fellner
2014-10-15 19:33 ` Ian Main
2014-10-17 16:14 ` Franz Fellner
2014-10-20 17:45 ` Ian Main
2014-10-20 19:05 ` Tomi Ollila
2014-10-20 19:38 ` Franz Fellner
2014-10-21 14:25 ` Tomi Ollila
2014-10-20 18:53 ` Tomi Ollila
2014-10-23 22:19 ` [PATCH v2] " Ian Main
2014-10-24 9:52 ` Tomi Ollila
2014-10-24 16:11 ` [PATCH v3] " Ian Main
2014-10-24 16:30 ` Tomi Ollila [this message]
2014-10-24 17:03 ` Ian Main
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m2oat177mi.fsf@guru.guru-group.fi \
--to=tomi.ollila@iki.fi \
--cc=imain@stemwinder.org \
--cc=notmuch@notmuchmail.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).