Re: [PATCH] org-protocol: Fix missing '+' in js snippet

[Maxim Nikulin]

On 19/04/2021 17:53, David Asabina wrote:
>   
>     javascript:location.href = \\='org-protocol://capture?url=\\='+ \\
> -        encodeURIComponent(location.href) + \\='&title=\\=' \\
> +        encodeURIComponent(location.href) + \\='&title=\\=' + \\
>           encodeURIComponent(document.title) + \\='&body=\\=' + \\
>           encodeURIComponent(window.getSelection())

I am not a maintainer, so I can not merge the patch. I hope, finally it 
will happen, maybe some months later.

I have a question that is a bit off-topic in this thread however. Since 
you are setting up org-protocol, you may notice other issues. There were 
reports that two slashes after "org-protocol:" cause some problems. My 
guess is that capture is recognized as hostname, so another slash is 
inserted after it. There was a patch allowing "/" before "?" to get 
things work (perhaps on Windows). Could you try, please, if one or three 
slashes in org-protocol URIs ("org-protocol:/capture?url=..." and 
"org-protocol:///capture?url=...") works properly in your environment? 
Maybe code examples in docs would benefit from one more correction.